Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When using multi-level drawers, the attribute overflow-y: hidden; appears on the body after all are closed. #28528

Closed
1 task
wellssu0 opened this issue Dec 24, 2020 · 23 comments · Fixed by react-component/drawer#157 or #28749

Comments

@wellssu0
Copy link
Contributor

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

https://ant.design/components/drawer/

Steps to reproduce

点击 Multi-level drawer 示例 的 open drawer 按钮 ,弹出后再点击 two-level drawer 按钮,弹出后将两层抽屉关闭,body上会出现overflow-y: hidden; 此时页面不能滚动.

What is expected?

body上没有overflow-y属性

What is actually happening?

body上加了overflow-y: hidden属性

Environment Info
antd 4.7.0
React 16.8.6
System macos 11.1
Browser chrome Version 87.0.4280.88

我自己的项目里出现了这个问题,又查看了官网的示例,发现官网的示例也存在这个问题

@wellssu0
Copy link
Contributor Author

请问多久可以修复这个问题呢?

@shaodahong
Copy link
Member

请问多久可以修复这个问题呢?

预计这个月的 minor

@demonskp
Copy link

遇到了相同的问题。貌似是rc-util的引用不同 引起的

@shaodahong
Copy link
Member

遇到了相同的问题。貌似是rc-util的引用不同 引起的

这个确实可能会,但是 rc-util 是没有锁 minor 版本的,按理说不应该会有多版本的问题

@demonskp
Copy link

遇到了相同的问题。貌似是rc-util的引用不同 引起的

这个确实可能会,但是 rc-util 是没有锁 minor 版本的,按理说不应该会有多版本的问题

遇到问题的项目,如果用npm安装 不会有问题最外层rc-util的版本是4.21.1 然后如果用yarn安装会有问题,是5.6.1 如果在项目里主动加一个rc-util 4.21.1的依赖 就没这个问题了

@shaodahong
Copy link
Member

rc-util 4.21.1?,antd 3 么

@demonskp
Copy link

rc-util 4.21.1?,antd 3 么

是4.X 但是不是具体的版本不确定了。 可能是其他引用的组件库 有antd3的内容

@shaodahong
Copy link
Member

如果是多版本有点无解,就和 React 多版本一样,也是跑不起来的,只能有一个实例,因为底层实现就是计数器,多版本多个计数肯定有问题

@demonskp
Copy link

好的 谢谢

@wellssu0
Copy link
Contributor Author

遇到了相同的问题。貌似是rc-util的引用不同 引起的

这个确实可能会,但是 rc-util 是没有锁 minor 版本的,按理说不应该会有多版本的问题

遇到问题的项目,如果用npm安装 不会有问题最外层rc-util的版本是4.21.1 然后如果用yarn安装会有问题,是5.6.1 如果在项目里主动加一个rc-util 4.21.1的依赖 就没这个问题了

今天项目本地测试是好的,我又在用yarn和npm分别重新装了node_modules,测试也都可以滚动,body上只有style了。但重新发到测试站上测试还是依然存在问题。

@shaodahong
Copy link
Member

jenkins 构建的么,jenkins 的话删除下工作空间试试

@wellssu0
Copy link
Contributor Author

jenkins 构建的么,jenkins 的话删除下工作空间试试

构建项目使用的是阿里云的云效Flow

@shaodahong
Copy link
Member

……没用过,本地好的线上不行听起来很像是缓存的问题

@wellssu0
Copy link
Contributor Author

……没用过,本地好的线上不行听起来很像是缓存的问题

构建的时候有清理工作区这一步。 官网示例上的这个问题还没解决吗

@ISBN9527
Copy link

+1今天刚碰到这个问题,查了老一会😓

@ISBN9527
Copy link

希望早点发版本😭😿

@ouzhou
Copy link

ouzhou commented Dec 28, 2020

我的报错了,也是关闭后无法滚动,并且dom节点上有明显的错误
devScripts.js:5836 Warning: React does not recognize the scrollLocker prop on a DOM element.
image

@ISBN9527
Copy link

还没解决吗?

@afc163
Copy link
Member

afc163 commented Dec 28, 2020

@zombieJ 看看要不要回滚 react-component/util#174

@wellssu0
Copy link
Contributor Author

……没用过,本地好的线上不行听起来很像是缓存的问题

今天仔细看了一下云效flow的设置 果然有缓存,把缓存选项去掉重新构建了一遍就正常了,现在测试站用的4.7.3版本。我本地测试过4.7.3,4.8.5,4.9.4都没有问题

@wellssu0
Copy link
Contributor Author

……没用过,本地好的线上不行听起来很像是缓存的问题

今天仔细看了一下云效flow的设置 果然有缓存,把缓存选项去掉重新构建了一遍就正常了,现在测试站用的4.7.3版本。我本地测试过4.7.3,4.8.5,4.9.4都没有问题

还有云效flow构建默认使用的cnpm改回了npm 不知道跟这个有没有关系

@shaodahong
Copy link
Member

cnpm改回了npm

没关系,当然了 yarn 就不好说,摊平方式不一样可能摊成多版本说不定,下次可以 yarn/npm list rc-util 看看本地依赖,如果本地有问题的话

@wellssu0
Copy link
Contributor Author

cnpm改回了npm

没关系,当然了 yarn 就不好说,摊平方式不一样可能摊成多版本说不定,下次可以 yarn/npm list rc-util 看看本地依赖,如果本地有问题的话

好的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants