processing/p5.js

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

Open

#5.228 geöffnet am 6. Mai 2021

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (3.178 Forks)batch import
Area:ImageBugHelp Wantedp5.js 2.0+

Repository-Metriken

Stars
 (20.784 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T 13h) (45 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Der Bug befindet sich im GIF Wiedergabecode, wahrscheinlich in src/image/p5.Image.js. Die Behebung beinhaltet, sicherzustellen, dass gifProperties.displayIndex niemals gifProperties.frames.length 1 überschreitet, bevor play() aufgerufen wird. Der Autor des Problems hat eine minimale Reproduktion im p5.js Editor bereitgestellt und schlägt eine einfache Grenzwertprüfung vor. Suchen Sie nach der play() Methode und validieren Sie displayIndex gegen die Länge von frames.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptp5.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90