davidtheclark/no-scroll

Previous styles on documentElement are not restored

Open

#10 aperta il 2 gen 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)HTML (13 fork)github user discovery
bughelp wanted

Metriche repository

Star
 (244 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I had a use case where the documentElement had inline styles on it, like so: <html style="padding: 0; margin: 0; overflow-x=hidden;">. I noticed that after toggling no-scroll to off, the overflow-x=hidden was being wiped out. After checking the code in no-scroll, I found that it's hard-coding the style attributes when off() is called. I think that no-scroll should be restoring whatever values were set before on() was called, instead of overwriting them.

Guida contributor

Direzione di ricerca
Ispeziona il codice sorgente della funzione off(); attualmente sovrascrive gli attributi di stile di documentElement. Modifica on() per memorizzare i valori originali degli attributi di stile e off() per ripristinarli.
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.
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80