-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Use <dialog> element #60
Comments
Moin @medienbaecker 👋, the native dialog element has also been on my list for some time. I think it's best to implement it with the next major release (3.0.0), which I plan to start next weekend. |
Version 3.0.0 is now available as a beta and will be released shortly after some intensive testing, thus closing this issue. |
Can confirm the dialog element is working fine in 3.0.0-beta! |
Hi @sciloqi, I just read your other comment and was about to try to reproduce it. Thank you for letting me know. |
Thanks for coming back to this, but all is ok. i accidentally switched to an old version when testing the beta, but both 3.0.0-alpha and 3.0.0-beta work fine, even if you open a lightbox from another dialog element. So, everything is perfect, thank you very much! 👍 |
Great job, it's working for me, too 👍 |
I finally had the chance to test Parvus today, and I love it — finally an accessible lightbox!
What do you think about using the dialog element instead of a
z-index
of1337
? :)Also: automatic ARIA role, focus management, keyboard navigation — I think it would be a great fit.
The text was updated successfully, but these errors were encountered: