-
Notifications
You must be signed in to change notification settings - Fork 130
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
Set aria-controls on opening button if dialog is not a direct sibling #113
Comments
Good morning Marco! Could I ask you to provide the reference in the specifications for this maybe? I would like to read a little bit about it before implementing it. Thank you! :) |
The first time I read about Here's the spec part: https://www.w3.org/WAI/PF/aria-1.1/states_and_properties#aria-controls There is a wonderful article by Heydon Pickering on why NOT to use it: http://www.heydonworks.com/article/aria-controls-is-poop Especially the thread on the Github issue is the point where I think it makes sense to add the attribute with a matching value of the ID on the dialog. Using Makes sense? |
I’m coming back on this and am a little on the fence. I don’t know whether a button opening a dialog should have the |
I’m going to close this as I’m struggling to find relevant documentation illustrating the usage of |
If the modal dialog is not a direct sibling of the opening button, this button needs an
aria-controls
attribute with a value matching the ID of the dialog.Currently I'd set this manually but maybe it's possible to do this programmatically.
The text was updated successfully, but these errors were encountered: