processing/p5.js

GIFs that don't loop break unless reset before play()

Open

#5 228 ouverte le 6 mai 2021

Voir sur GitHub
 (11 commentaires) (0 réactions) (1 assigné)JavaScript (3 178 forks)batch import
Area:ImageBugHelp Wantedp5.js 2.0+

Métriques du dépôt

Stars
 (20 784 stars)
Métriques de merge PR
 (Merge moyen 8j 13h) (45 PRs mergées en 30 j)

Description

Most appropriate sub-area of p5.js?

  • Accessibility (Web Accessibility)
  • Build tools and processes
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Friendly error system
  • Image
  • IO (Input/Output)
  • Localization
  • Math
  • Unit Testing
  • Typography
  • Utilities
  • WebGL
  • Other (specify if possible)

Details about the bug:

  • p5.js version: 1.0+
  • Web browser and version: all
  • Operating System: all
  • Steps to reproduce this: Here is an example. Comment out the gif.setFrame(0) to see the crash and unfriendly error log.

I think it is okay to expect the coder to manually reset to the beginning of a non-looping GIF before playing but when they don't do that it shouldn't crash. A check to make sure that gifProperties.displayIndex never exceeds gifProperties.frames.length - 1 should do the trick!

This could be a nice issue for a newer contributor. I am happy to help.

Guide contributeur

Direction de recherche
Le bug se trouve dans le code de lecture GIF, probablement dans src/image/p5.Image.js. La correction consiste à s'assurer que gifProperties.displayIndex ne dépasse jamais gifProperties.frames.length 1 avant d'appeler play(). L'auteur du problème a fourni une reproduction minimale dans l'éditeur p5.js et suggère une simple vérification des limites. Cherchez la méthode play() et validez displayIndex par rapport à la longueur de frames.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptp5.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90