-
Notifications
You must be signed in to change notification settings - Fork 808
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
Focus escapes modal when radio buttons in modal #636
Comments
I think the issue here is that radio buttons are considered individually focusable by |
I've also noticed that if you hold your mouse down on a radio button then focus is shifted back to I'm not sure if this a related or separate issue. |
We ran into a similar issue. Here are some details. Our modal contains a bunch of fieldsets. Each fieldset has several radio buttons. When you press tab on chrome, it only cycles through just the first radio button within each fieldset. Whereas on Safari, it goes to each radio button for each fieldset. As a result, when we press tab to reach the first radio of the last fieldset, and then press tab again, it does not cycle to the modal close button because the first option of the last fieldset is not the tail element in the list of tabbable elements. Here's where the logic fails. |
Hello, I don't mean to necro this, but has anyone found a workaround for this issue? I noticed that this issue is still open and I, too, am experiencing this issue :( |
🧟 Bringing this back form the dead has there been any work on finding a solution? The current solution is clearly a WCAG violation as it breaks navigation for keyboard users, at minimum the documentation should be updated to highlight this. |
@stephenmcm @coulgreer @ryansukale The issue is still open. It's up for grabs - PRs are welcome. |
Hey @diasbruno, I'm going to take another look into this one. 👀 Will update once I have more information! |
@doeg Awesome. Let me know if you need anything. |
Summary:
When there are radio buttons in a modal, you can escape focus with shift-tab.
Steps to reproduce:
Steps are included in the test case below
Expected behavior:
You should not be able to escape the modal's focus
Link to example of issue:
https://codesandbox.io/s/my9vm34m68
Additional notes:
Not sure if it's related specifically to radio buttons, but that is what is causing the issue for me
The text was updated successfully, but these errors were encountered: