streamich/react-use

useLockBodyScroll doesn't restore scroll position

Open

#840 aperta il 16 dic 2019

Vedi su GitHub
 (7 commenti) (0 reazioni) (1 assegnatario)TypeScript (3273 fork)batch import
bughelp wanted

Metriche repository

Star
 (43.979 star)
Metriche merge PR
 (Merge medio 1743g 9h) (1 PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il bug si trova nell'hook useLockBodyScroll in react use. Per risolverlo, è necessario salvare la posizione di scorrimento quando lo scorrimento è bloccato e ripristinarla quando viene sbloccato. Esamina l'implementazione dell'hook in https://github.com/streamich/react use/blob/master/src/useLockBodyScroll.ts. Il codice attuale probabilmente imposta overflow hidden ma non memorizza/ripristina scrollLeft/scrollTop. Rivedi i commenti esistenti e il lavoro dell'assegnatario per evitare duplicazioni.
Tech stack
reacttypescript
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactDOM manipulation
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65