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

Modal role="alertdialog" #2701

Closed
shaunnbarron opened this issue Aug 24, 2023 · 1 comment · Fixed by #2709
Closed

Modal role="alertdialog" #2701

shaunnbarron opened this issue Aug 24, 2023 · 1 comment · Fixed by #2709
Assignees
@thecrypticace

Comments

@shaunnbarron
Copy link

shaunnbarron commented Aug 24, 2023 •
edited

What package within Headless UI are you using?

@headlessui/react and @headlessui/vue

What version of that package are you using?

v1.7.16

What browser are you using?

Chromium

Reproduction URL

https://stackblitz.com/edit/nuxtlabs-ui-6njwbz?file=app.vue

Describe your issue

Unable to create a Modal with an alertdialog role in order to implement the Alert Dialog pattern.
@thecrypticace thecrypticace self-assigned this Aug 25, 2023
@thecrypticace thecrypticace mentioned this issue Aug 28, 2023
Merged
@thecrypticace
Copy link
Contributor

Hey, thanks for brining this one up. We've added support for this in #2709, and it'll be available in the next release.

You can give this a try already by using our insiders build (it may take a few minutes to publish to NPM):

  • npm install @headlessui/react@insiders.
  • npm install @headlessui/vue@insiders.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thecrypticace thecrypticace

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for role="alertdialog" to <Dialog> component tailwindlabs/headlessui
2 participants
@thecrypticace @shaunnbarron