-
Notifications
You must be signed in to change notification settings - Fork 361
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
Disclosure Menu Examples have conflicting behavior and descriptions for the Esc
key
#2830
Comments
The ARIA Authoring Practices (APG) Task Force just discussed The full IRC log of that discussion<jugglinmike> Topic: Escape in disclosure nav menu<jugglinmike> github: https://github.com//issues/2830 <jugglinmike> Matt_King: Escape is all optional <jugglinmike> Matt_King: I don't like the way we use the word "Dropdown" here. It doesn't seem like the right word in a disclosure because it sort of implies a menu <jugglinmike> Matt_King: Why do we close these on blur in the disclosure? That doesn't seem right <jugglinmike> Matt_King: Going back to our previous discussion, if you tab out, you should be able to tab back to where you were <jugglinmike> Matt_King: This is not a menu, so tabbing is not expected to close. If you have a link, for example, and you tab away from that link, you should be able to tab back to that link. I'd be shocked if I couldn't do that <jugglinmike> CoryJoseph: What the reporter is describing doesn't seem like a bug according to the definition of Disclosure Menu <jugglinmike> jugglinmike: I wonder if there's a parallel with mouse users. If I hover out of the dropdown, it remains open. To the extent that the position of my cursor describes my "focus" as a mouse user, then the "close on blur" behavior seems inconsistent for keyboard users <jugglinmike> jongund: It seems like when one menu opens, it should close the other menus. One reason people use menus is to reduce visual clutter <jugglinmike> Matt_King: That seems reasonable to me, but you have to take an explicit action to expand the other menu <jugglinmike> Matt_King: The issue that was reported appears to be editorial, but I'm having trouble understanding as written <jugglinmike> Matt_King: I'll ask for clarification <jugglinmike> Matt_King: There's also a potential bug here which might be a separate issue, but we'll see about that after we receive clarification <jugglinmike> Zakim, end the meeting |
Re-reading this after our meeting discussion today, I think I understand your concern. Would it address your concern if we were to change the accessibility features text to the following?
|
This would address the issue of documentation I think, yes. There was this second issue I mentioned that I either need clarification on or I would assume it was a bug:
The top-level links are in the navigation region, based on the documentation I would expect that if a disclosure menu was open, and focus was on a top-level link within the navigation region, when "Escape" was pressed the open disclosure menu would also close. Is this an incorrect interpretation? Currently, the disclosure menu stays open if your focus is not in the disclosure, but on a top-level link when you press "Escape". |
In Example Disclosure Navigation Menu we can see the following under "Accessibility Features":
Then under "Keyboard Support" we can see the following behavior described for the "Escape" key:
Both described behaviors seem to be implemented, but I'm not sure why the description of the two differing behaviors based on where focus currently is is split up in these two locations, if someone read one and not the other, they'd miss an implementation detail.
Secondly, the same things are true about Example Disclosure Navigation Menu with Top-Level Links however there is a novel bug in the implementation of the example, where if focus is on one of the top level links pressing "Escape" does nothing at all, this seems to be in conflict with the description of what should happen if you press escape while in the navigation region.
The text was updated successfully, but these errors were encountered: