GoogleChromeLabs/quicklink

Consider adding `rootMargin` to the Intersection Observer

Open

#14 aperta il 10 dic 2018

Vedi su GitHub
 (13 commenti) (1 reazione) (0 assegnatari)JavaScript (431 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (10.861 star)
Metriche merge PR
 (Merge medio 47g 12h) (4 PR mergiate in 30 g)

Descrizione

Feedback from @surma:

I feel like by default you should add a rootMargin to the IO to trigger the load before something comes into the viewport.

So, rootMargin defaults to "0px 0px 0px 0px", meaning the intersection will be computed between the root element's unmodified bounds rectangle and the target's bounds. This change would be made to https://github.com/GoogleChromeLabs/quicklink/blob/master/src/index.mjs#L21

Guida contributor

Direzione di ricerca
Modifica la chiamata al costruttore IntersectionObserver in src/index.mjs per accettare un parametro rootMargin opzionale, con default '0px 0px 0px 0px', e aggiorna di conseguenza la firma della funzione e la documentazione.
Tech stack
javascript
Dominio
frontend
Tipo issue
Funzionalità
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
JavaScriptIntersection Observer API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70