Bug 527884 - Local Editor Settings popup needs accessibility work
Summary: Local Editor Settings popup needs accessibility work
Status: NEW
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Client (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2017-11-28 15:10 EST by Carolyn MacLeod CLA
Modified: 2017-11-28 15:10 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2017-11-28 15:10:03 EST 
The Local Editor Settings (wrench) has a "DropDownMenu" that needs to be marked up for accessibility. Unfortunately, we can't give it role="menu", because in the accessibility world, the contract for a menu is that it has menu items, and those can be buttons or checkboxes or radios, but they can't be combos or links.

I think the most accessible role for this is a "popup dialog", which would be marked up by putting aria-popup="dialog" on the wrench button, and role="dialog" on the div with id="_navdropdown" that is created on line 445 of editorCommands.js. The aria-label on the _navdropdown would be "Local Editor Settings". Need to add down arrow support on the button to show the popup.

Not sure if this support is needed by other callers of new DropDownMenu(...).
See line 458 in editorCommands.js. If yes, then add a parameter to DropDownMenu initialization to set this up.