-
Notifications
You must be signed in to change notification settings - Fork 22
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
Ambiguous but normative requirement about hidden nodes is hidden by default in AccName #211
Comments
Maybe add "aria-hidden project"? |
I mispoke on the call. A "hidden" project would include more than just aria-hidden: inert, css visibility, display:none, content-visibility, etc. |
…but the issues span across various specs: html (main spec), html-aam, css, aria, accname, etc. I'm not sure we can make a project in the aria repo that cross-references non-aria-spec issues. |
Discussed in today's meeting: https://www.w3.org/2023/11/02-aria-minutes.html#t09 Missed some of the discussion in the meeting, but the next steps (as I understand them) are:
Separately, @jnurthen will open an editorial PR to take notes out of the summary details format. |
FWIW, I think it's okay to have extended notes and examples in a collapsed |
@cookiecrook @jnurthen @spectranaut Following up on this:
Done, here is the WPT test PR: web-platform-tests/wpt#43043.
It looks like the current accName 2B note for labelledby, as written, is broadly implemented across browsers (except for some WebKit |
Follow-on from #209.
If you expand the "Comment" details/summary disclosure in comp_labelledby, there is a hidden (ironic?) RFC-2119 requirement:
I would first suggest that it's a bad idea to hide normative requirements by default.
Second, I'm not sure what this normative requirement means. "MUST include all nodes" seems like it's requiring all the "if hidden" considerations in the algorithm to be ignored, and all DOM subtree text must be included in the accessible name. But that can't be right.
I think this requirement might be contradictory to the more recent momentum around hidden:
We probably just didn't notice it because it was hidden... 🤣
The text was updated successfully, but these errors were encountered: