streamich/react-use

useLockBodyScroll doesn't restore scroll position

开放

#840 创建于 2019年12月16日

在 GitHub 查看贡献者指南
 (7 条评论) (0 个反应) (1 位负责人)TypeScript (3,273 个派生)batch import
bughelp wanted

仓库指标

星标
 (43,979 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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

贡献者指南

研究方向
bug 位于 react use 中的 useLockBodyScroll 钩子。修复时，需要在滚动被锁定时保存滚动位置，并在解锁时恢复。查看 https://github.com/streamich/react use/blob/master/src/useLockBodyScroll.ts 中的钩子实现。当前代码可能设置了 overflow hidden，但没有存储/恢复 scrollLeft/scrollTop。请查看现有评论和 assignee 的工作，避免重复。
技术栈
reacttypescript
领域
frontend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
ReactDOM manipulation
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65