davidtheclark/no-scroll

Previous styles on documentElement are not restored

Open

#10 ouverte le 2 janv. 2019

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)HTML (13 forks)github user discovery
bughelp wanted

Métriques du dépôt

Stars
 (244 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Inspectez le code source de la fonction off() ; elle écrase actuellement les attributs de style de documentElement. Modifiez on() pour stocker les valeurs d'attribut de style d'origine et off() pour les restaurer.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80