-
Notifications
You must be signed in to change notification settings - Fork 533
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
Dialog: Support a responsive value for isOpen
#4755
Comments
isOpen
isOpen
Notes from Maintainer Sync (July 22nd)Context from Slack: https://github.slack.com/archives/GACAW0NPM/p1721383080023129 We're going to leave this in the inbox this week while waiting for the conversation in Slack to resolve 👀 |
Notes from Maintainer Sync (August 5th)It might be a good idea to experiment / prototype how this would work, testing focus management, declarative buttons of a Dialog, etc. We should revisit this in a future planning cycle to see if we can dedicate some time to the explorations mentioned. Please let us know if there is any urgency around this sooner. |
Description
In some cases you want to display data in a dialog on mobile but on desktop keep it on the page.
This is currently not possible with this component due to the lack of responsive support on the isOpen value.
Wrapping the dialog in a div and setting
display: none
on smaller breakpoints neither works.Steps to reproduce
Version
Latest
Browser
Safari
The text was updated successfully, but these errors were encountered: