Closed Bug 977244 Opened 9 years ago Closed 3 years ago
[a11y] style inspector [rule view] needs some a11y love
59 bytes, text/x-review-board-request
47 bytes, text/x-phabricator-request
|Details | Review|
The Style Inspector aka Rule View right now is not very useful from an a11y standpoint. It presents itself as some groups of lists with rule names and possibly no values. One suggestion could be to convert these lists into rows of labels and textboxes. Alternately, we could add aria-labels to the rules and values. Keyboard navigation through the rule view is currently impossible.
This is a larger project and not done in five minutes. Setting to P2 for now.
Priority: -- → P2
This is still an outstanding issue. What we want is to add the necessary semantics to the rules view. It can either be a list/listbox or tree view semantics. I believe firebug has a tree.
Component: Developer Tools: Inspector → Developer Tools: CSS Rules Inspector
Assignee: nobody → yzenevich
Status: NEW → ASSIGNED
Comment on attachment 8831381 [details] Bug 977244 - [a11y] Style inspector [rule view] needs some a11y love. https://reviewboard.mozilla.org/r/107952/#review109786 I believe you mentioned you wanted some early feedback. Sorry if I remembered wrong :) * Current section should be scrolled into view when arrowing up and down. There is a utility in devtools that does that I believe. * When you use role group you can't really add anything to the role value (e.g. index). Simply role="group" * We definitely need focus for groups * Have to tab twice to get away from the rule view container (I think it's some scrollable thing that is also focusable) * When entering a group, we get into CSS rule edit mode. Is it possible to just focus instead of activating an input? * Focused checkbox needs style * Focused expand trianle needs style * When in an expandable row, the whole row should listen for left/right keys to collapse/expand the style. * Probably not everything in the group should be a tree item. I think only the things that are list items should. I think this is a good start :) ::: devtools/client/inspector/inspector.xhtml:102 (Diff revision 1) > <div id="ruleview-command-toolbar"> > <button id="ruleview-add-rule-button" data-localization="title=inspector.addRule.tooltip" class="devtools-button"></button> > <button id="pseudo-class-panel-toggle" data-localization="title=inspector.togglePseudo.tooltip" class="devtools-button"></button> > </div> > </div> > <div id="pseudo-class-panel" hidden="true">
Attachment #8831381 - Flags: review?(yzenevich)
Assignee: npang → nobody
Status: ASSIGNED → NEW
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/be296969313c Make the rules view easier to navigate with a screen reader, r=yzen
You need to log in before you can comment on or make changes to this bug.