-
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
2E "unless the element is marked as presentational" is unclear #108
Comments
@JAWS-test do you think these issues are a side effect of invalid code, or did the implementors genuinely not understand the spec? |
@MelSumner Neither nor. If the specification is unclear, every browser does what it thinks. The specification should be clear, then the browsers know how to do it right. |
Updated the title due to the re-ordering in #122 |
+1 This issue is an older one, but I believe it still needs to be addressed. The behavior in these cases is unclear:
I would expect the accessible name to remain "First Name" in both scenarios, but the specifications are not clear on this. The current wording, particularly the phrase "unless the element is marked...", is ambiguous. It is unclear whether this refers to the "current node" or to the "element (e.g., an HTML label or SVG title)". To resolve the ambiguity, changing it to "unless the current node is marked..." would provide clarity. There is still one issue to address regarding the Presentational Roles Conflict Resolution. Even if I’m opening a tentative PR to address these points. |
I think we need to talk about this in the WG meeting. I'm not convinced this is an AccName specific issue since the application of the presentational role doesn't just impact the name but also the role, and any focusable element has its presentational state ignored. |
FWIW i agree with @giacomo-petri's assumption that even if a similarly, i'd also expect (which matches reality) that the name of this fieldset is "foo" |
It's strange behavior to have so many exceptions just because developers make so many mistakes. What is the point of having the role of presentation or none if it only works sometimes? We're twisting ourselves into pretzels to accommodate code that is simply invalid. We need a better solution than this. |
@MelSumner i don't see this as adding exceptions. there are differences in behavior here, as i mentioned by jaws test, and further called out in the back and forth i was having with Giacomo in his PR for this issue. so it's not unreasonable to to get this clarified, since there isn't full interop. i don't disagree that some of this could be considered invalid code from the authoring side - but the ARIA spec is clear that unless a host language explicitly states that certain features are not to be overwritten by ARIA, or ARIA doesn't have other stipulations written, then ARIA takes precedent. additionally, this isn't about making but if role none is put on the "current node", as is one interpretation of the spec text (which i think is probably the right one) - THEN the role=none kicks in and the element shouldn't be named, because it can't be, it no longer has a role that allows it to be named. so role=none working as it should. no twisting. TBH, where there is a problem here, it's not even unique to with that said, i think @giacomo-petri has done a good job at trying to address this issue. i had some confusion about what the expected outcome should be due to the wpt that was initially made, but after clearing that up, it looks like this was on the right track and it'll make things more consistent / keep things accessible even if authors do something weird, once this has been finalized. |
In my opinion, the following problems arise here:
This is problematic because this is currently interpreted differently by browsers and AccName Prototype Test (see the examples)
The text was updated successfully, but these errors were encountered: