jquery/jquery

Rearrange animation options logic

Open

#2255 aperta il 3 mag 2015

Vedi su GitHub
 (16 commenti) (0 reazioni) (0 assegnatari)JavaScript (20.438 fork)batch import
Effectshelp wanted

Metriche repository

Star
 (59.842 star)
Metriche merge PR
 (Merge medio 16h 54m) (7 PR mergiate in 30 g)

Descrizione

cc @gnarf

.animate uses the undocumented jQuery.speed (its only use, in fact) to manipulate options before sending them to Animation, where they are preserved as "original": https://github.com/jquery/jquery/pull/2218#issuecomment-98504382

We should move such manipulation so that anim.originalOptions is in fact what it claims to be.

Guida contributor

Direzione di ricerca
Studia come `jQuery.speed` manipola le opzioni e come `Animation` memorizza `originalOptions`. Proponi di spostare la manipolazione in `Animation` o in una funzione separata in modo che `originalOptions` rifletta l'input dell'utente. Traccia il percorso del codice da `.animate()` al costruttore di `Animation`.
Tech stack
javascript
Dominio
frontend
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptjQuery internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65