streamich/react-use

useLockBodyScroll doesn't restore scroll position

Open

#840 geöffnet am 16. Dez. 2019

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (1 zugewiesene Person)TypeScript (3.273 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (43.979 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1743T 9h) (1 gemergte PR in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Der Bug befindet sich im useLockBodyScroll Hook in react use. Zur Behebung müssen Sie die Scroll Position speichern, wenn das Scrollen gesperrt ist, und sie bei Entsperrung wiederherstellen. Sehen Sie sich die Hook Implementierung unter https://github.com/streamich/react use/blob/master/src/useLockBodyScroll.ts an. Der aktuelle Code setzt wahrscheinlich overflow hidden, speichert/stellt aber scrollLeft/scrollTop nicht wieder her. Überprüfen Sie vorhandene Kommentare und die Arbeit des Assignees, um Duplizierung zu vermeiden.
Tech Stack
reacttypescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ReactDOM manipulation
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65