streamich/react-use

useLockBodyScroll doesn't restore scroll position

Open

#840 ouverte le 16 déc. 2019

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)TypeScript (3 273 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (43 979 stars)
Métriques de merge PR
 (Merge moyen 1743j 9h) (1 PR mergée en 30 j)

Description

What is the current behavior?

When useLockBodyScroll is passed a value of false, it does re-enable scrolling, however any previous scroll position is not restored.

What is the expected behavior?

The previous scroll position should be restored to the scrolled element.

A little about versions:

  • OS:
  • Browser (vendor and version):
  • React: 16.11.0
  • react-use: 13.10.1

Guide contributeur

Direction de recherche
Le bug se trouve dans le hook useLockBodyScroll de react use. Pour le corriger, vous devez enregistrer la position de défilement lorsque le défilement est verrouillé et la restaurer lorsqu'il est déverrouillé. Consultez l'implémentation du hook sur https://github.com/streamich/react use/blob/master/src/useLockBodyScroll.ts. Le code actuel définit probablement overflow hidden mais ne stocke/restaure pas scrollLeft/scrollTop. Examinez les commentaires existants et le travail de l'assigné pour éviter les doublons.
Stack technique
reacttypescript
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactDOM manipulation
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65