streamich/react-use

useLockBodyScroll doesn't restore scroll position

Aberta

#840 aberto em 16 de dez. de 2019

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (1 responsável)TypeScript (3.273 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (43.979 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
O bug está no hook useLockBodyScroll do react use. Para corrigir, você precisa salvar a posição de rolagem quando a rolagem está bloqueada e restaurá la quando desbloqueada. Veja a implementação do hook em https://github.com/streamich/react use/blob/master/src/useLockBodyScroll.ts. O código atual provavelmente define overflow hidden, mas não armazena/restaura scrollLeft/scrollTop. Revise os comentários existentes e o trabalho do assignee para evitar duplicação.
Pilha de tecnologia
reacttypescript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ReactDOM manipulation
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65