-
Notifications
You must be signed in to change notification settings - Fork 27
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
summary element role mapping #345
Comments
I thought we had previously discussed that the button mapping for summary is broken because summary can have interactive children in HTML |
i mean yes... there's very much that as well. but is that a spec issue and HTML should update to not allow interactive elements within? Or is that a potential implementation or mapping issue and we address this that way? |
I don't see HTML changing that. There are far too many implementations which meet the spec today which would become non-conformant. I think we need to work out how we should map summary in a useful way before adding an ARIA mapping for it. |
yeh, re: HTML... I understand that, but it just seems very unfortunate to not want to break something (by changing allowances) which, by current functionality, is already broken because of those permissive allowances. agenda or deep dive topic, @jnurthen ? |
Perhaps it would be useful to point out the problem in the "Comment" row of the summary table. This seems to make sense to me, because the HTML specification refers to HTML-AAM via the link "For implementers" |
It doesn't appear that there is 0 support for changing the HTML spec. This is the classic nested interactives debate. It is hard to imagine a solution that doesn't either:
I'm keen to see what people can come up with that solves the AX issues without disallowing interactives inside summary. |
Wild. In addition to causing accessibility issues, this sure makes keyboard usage weird for both sighted keyboard users and screen reader users. For example, below I have a summary that contains a link. Focusing the summary allows space and enter to expand/collapse the details. Focusing the the link and typing enter follows the link, but space on the link does nothing in Firefox and expands/collapses the details in Chrome (heh, in Chrome there's a bug where space does a page scroll as well, i.e. the event needs stopPropagation()/preventDefault() ). So the sighted keyboard behavior is not terrible, I suppose. But it gets weirder when using a screen reader. NVDA/Chrome/FF always follows the link on space and enter even when the summary has focus. JAWS/Chrome always expands/collapses on space and enter no matter what has focus. I had to use the context menu (Shift+F10) to follow the link. JAWS/FF works similar to regular keyboard, where focusing summary expands/collapses on space/enter and focusing link and typing enter follows link (space does nothing). (Sorry about the accidental comments and closing - I somehow managed to do that while testing space in JAWS/Edge). |
removing the agenda label for now so this stops coming up on the agenda item list. will add it back on when i think we have new ground to cover. |
From the ARIA WG deep dive today, there are presently two directions we could take this issue.
Thank you everyone for your participation / input in today's meeting. |
adding agenda so that this can be mentioned during the next wg meeting, to ensure people are aware of the choices / can comment on the issue. THEN, we can give people some time to make a decision, and in a couple/few weeks time we can take an action on the choice that we make. |
2¢ I feel like perhaps we need both 1 and 2. 1 because the role mapping for summary element is currently a mess across AT and browsers. 2 to solve the issue with interactive content inside of summary. And lastly, (perhaps the least prioritized?) with regards to the accordion proposal potentially adding the name attribute, something else might be added/considered to resolve the accname for the shadow button element. |
we have no ARIA mapping for the summary element. should we mention the ARIA equivalent being
role=button
witharia-expanded
?The text was updated successfully, but these errors were encountered: