Skip to content
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

Missing screen reader for button with menus #2162

Closed
ebruchez opened this issue Apr 3, 2015 · 0 comments

Comments

@ebruchez
Copy link
Collaborator

@ebruchez ebruchez commented Apr 3, 2015

Including:

  • repeated grid row menu button
  • repeated section row menu button
  • home page menu buttons

A button should indicate that it has an associated menu, and the menu should read its items.

@ebruchez ebruchez self-assigned this Apr 3, 2015
@ebruchez ebruchez added this to the 4.9 milestone Apr 3, 2015
@ebruchez ebruchez closed this in eab78ad Apr 3, 2015
ebruchez added a commit that referenced this issue Apr 3, 2015
- support xf:trigger/@ROLE to place role on generated control
- patch bootstrap-dropdown.js to add/remove aria-expanded (already done in Bootstrap 3)
- update Home page, grid, section, and repeater
  - use aria-expanded
  - use role="menuitem|presentation"
ebruchez added a commit that referenced this issue Apr 4, 2015
ebruchez added a commit that referenced this issue Apr 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.