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.

Sign up for GitHub

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

Jump to bottom

When Drawer is open you can scroll the body of main content. #28391

Closed
1 task done
p-efimenko opened this issue Dec 16, 2020 · 8 comments · Fixed by react-component/drawer#157 or #28749
Closed
1 task done

When Drawer is open you can scroll the body of main content. #28391

p-efimenko opened this issue Dec 16, 2020 · 8 comments · Fixed by react-component/drawer#157 or #28749
Labels
🐛 Bug Ant Design Team had proved that this is a bug.

Comments

@p-efimenko
Copy link

p-efimenko commented Dec 16, 2020 •
edited

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

Reproduction link

Edit on CodeSandbox

Steps to reproduce

Just open the Drawer and scroll down.

What is expected?

overflow for body must be hidden

What is actually happening?

The main content under mask is scrolled.

Environment Info
antd 4.9.4
React 17.0.1
System Win 10
Browser Google Chrome 87.0.4280.88
@Tesla0916 Tesla0916 mentioned this issue Dec 16, 2020
Closed
1 task
@afc163
Copy link
Member

afc163 commented Dec 17, 2020

Broken by react-component/util#174

@afc163 afc163 mentioned this issue Dec 17, 2020
Merged
@afc163 afc163 mentioned this issue Dec 17, 2020
Merged
@ouzhou
Copy link

ouzhou commented Dec 28, 2020

image
4.9.4仍然有这个错误
4.9.5修复吗

@yoyo837
Copy link
Contributor

yoyo837 commented Dec 28, 2020

等待升级依赖。

@shaodahong shaodahong mentioned this issue Dec 30, 2020
Merged
This was referenced Jan 5, 2021
Closed
Merged
@123456wjw
Copy link

image
4.9.4仍然有这个错误
4.9.5修复吗

请问解决drawer遮挡不住body的滚动条 并且有打开drawer有scrollLocker这个waring的问题了嘛？

@yoyo837
Copy link
Contributor

yoyo837 commented Jan 6, 2021

没有，等发布

@afc163 afc163 mentioned this issue Jan 7, 2021
Merged
19 tasks
@topseczbw
Copy link

这个问题解决了吗？

1 similar comment
@nianyi-778
Copy link

这个问题解决了吗？

@mvshvets
Copy link

没有，等发布

In what release will there be a solution?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 Bug Ant Design Team had proved that this is a bug.
Projects
None yet
Milestone
No milestone
8 participants
@afc163 @yoyo837 @ouzhou @p-efimenko @nianyi-778 @topseczbw @123456wjw @mvshvets