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

Scrollable Dialog outside of content #10656

Closed
dlcoffee opened this issue Mar 15, 2018 · 3 comments
Closed

Scrollable Dialog outside of content #10656

dlcoffee opened this issue Mar 15, 2018 · 3 comments
Labels
duplicate This issue or pull request already exists

Comments

@dlcoffee
Copy link

I have a Dialog that is long, vertically, and autoScrollBodyContent={true} set, so that the content scrolls.

What I'd like to have happen is that when using the scrollbar/trackpad/mobile device to scroll outside of the Dialog, the content area also scrolls so that the user is not forced to target the content area.

Is this functionality achievable with any version of material-ui, or is this a pattern that should be avoided? I've tried adding event listeners to scroll, it doesn't get fired because the underlying page isn't scrollable due to Dialog being open.

@oliviertassinari
Copy link
Member

@dlcoffee Is it a duplicate of #7438?

@dlcoffee
Copy link
Author

@oliviertassinari yeah i think it's very similar. the solutions in that issue are for mobile only and suggest using a full screen Dialog instead. I was wondering if this would be possible for a regular sized Dialog on all devices. Maybe I should close this issue and comment on that one instead?

@oliviertassinari oliviertassinari added the duplicate This issue or pull request already exists label Mar 15, 2018
@oliviertassinari
Copy link
Member

@dlcoffee Alright, let's move the discussion to the other issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants