-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Auto focus the delete confirm button #5813
Labels
Comments
in which file is the confirm button located? |
After taking a look at the code, it seems that the dialog from radix ui uses the enter key to close the modal instead of confirming it.
https://www.radix-ui.com/primitives/docs/components/dialog#keyboard-interactions |
Not sure about this UX. What do you think @hwangdev97 ? |
lgtm 🤔 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
It would be better if the delete button in the confirmation modal is autoFocussed so that I can use the enter key instead of clicking the button to delete
Use case
No response
Anything else?
No response
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: