developit/decko

@debounce is named improperly — should be @throttle

Open

#9 aperta il 29 mar 2017

Vedi su GitHub
 (4 commenti) (10 reazioni) (0 assegnatari)JavaScript (34 fork)github user discovery
bugdiscussionhelp wanted

Metriche repository

Star
 (1037 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

In the docs its says:

To debounce means "call this at most once per N ms". All outward function calls get collated into a single inward call, and only the latest (most recent) arguments as passed on to the debounced function.

What you're doing is throttling, not debouncing. lodash.throttle:

Creates a throttled function that only invokes func at most once per every wait milliseconds.

lodash.debounce:

Creates a debounced function that delays invoking func until after wait milliseconds have elapsed since the last time the debounced function was invoked.

And while on the topic, a (real) @debounce would be a great addition.

Guida contributor

Direzione di ricerca
Rinomina il decoratore @debounce in @throttle nel codice sorgente, nei test e nella documentazione. Assicurati che l'implementazione esistente implementi correttamente il throttling. Opzionalmente, implementa un nuovo decoratore @debounce che ritarda l'invocazione fino a quando non sono trascorsi wait millisecondi dall'ultima invocazione.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75