streamich/react-use

useLockBodyScroll doesn't restore scroll position

Open

#840 创建于 2019年12月16日

在 GitHub 查看
 (7 评论) (0 反应) (1 负责人)TypeScript (3,273 fork)batch import
bughelp wanted

仓库指标

Star
 (43,979 star)
PR 合并指标
 (平均合并 1743天 9小时) (30 天内合并 1 个 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