TeamNewPipe/NewPipe

Standardize view animation duration

Open

#3038 aperta il 30 gen 2020

Vedi su GitHub
 (11 commenti) (3 reazioni) (0 assegnatari)Java (3577 fork)batch import
GUIfeature requestgood first issue

Metriche repository

Star
 (38.153 star)
Metriche merge PR
 (Merge medio 9g 17h) (17 PR mergiate in 30 g)

Descrizione

If you check the usages of animateView you will see that the typical range will be 0 all the way to 500, with most in the 100-250 range.

It should've been done with a default value in the first place. It is pretty much arbitrary right now, it was whatever the person felt like at the time of writing.

I'll leave this task to another PR, which the goal will be to standardize the used duration of animations.

_Originally posted by @mauriciocolli in https://github.com/TeamNewPipe/NewPipe/pull/2309#discussion_r371625549

Guida contributor

Direzione di ricerca
Identifica tutti gli usi di animateView nel codebase, categorizza i valori di durata e proponi una durata predefinita standardizzata o un insieme di costanti. Quindi implementa la modifica in tutte le occorrenze.
Tech stack
java
Dominio
mobile
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Android developmentGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70