-
Notifications
You must be signed in to change notification settings - Fork 127
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
Handling Author Errors: form & region roles #1683
Conversation
closes #1083 adds initial draft content to the handling author error section to call out expected behavior for `form` and `region` landmark roles which have not been provided accessible names.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For form
, this looks like a substantive change from what happens currently. It looks like <div role="form">
is still exposed as a form landmark even without a name (checked Firefox and Chromium on this quick example: https://jsfiddle.net/py96hL7b/, and Safari appears to map to Group
whether named or not).
Is that something we actually want to change? Out of all landmarks with name required, I can most see form
being valuable to expose anyway.
Fair point. The intent here was to align how HTML AAM treats unnamed If we do not want to make this change then this is really just to better specify what happens with |
To be fair, aligning to an unnamed |
Adding agenda label to hopefully get this quickly resolved. Solutions - keep PR as is if people agree that an unnamed Alternative, we decide that |
TODO: make sure that this PR indicates that the fallback role for the element is what is to be exposed AND if someone were to do |
@MelSumner I have finished my TODO, so this is ready to review again. @cookiecrook I added you as a reviewer as it was your comment (if i recall correctly) that prompted me to add the new bits to this PR. If you do not wish to review, please ignore. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@scottaohara that sounds like a good idea to me 👍 |
thanks @smhigley. added some words to those lines for each role. |
I agree with this change. If a form exists, it probably represents the purpose of the page and AT users should at least understand they are interacting with a form, even if it is unnamed. Anonymous regions aren't perfect but they still can provide a sense of structure for AT users. |
closes #1083 * adds initial draft content to the handling author error section to call out expected behavior for `form` and `region` landmark roles which have not been provided accessible names. * update section to talk about fallback and implicit role * add mention of naming to region/form landmark sentence
closes #1083
adds initial draft content to the handling author error section to call out expected behavior for
form
andregion
landmark roles which have not been provided accessible names.Diff