Ticket #3553 (new bug)

Opened 22 months ago

Last modified 8 months ago

Accordion needs keyboard and ARIA support

Reported by: davidb Owned by: davidb
Priority: major Milestone: 1.next
Component: [meta] ui.a11y Version: 1.6rc2
Keywords: Cc: michelled

Description

Currently accordion does not support a keyboard or screen reader interaction. We should fix this here so that jQuery users can benefit. This should probably be high priority given that it is locking out some features of the Mozilla addons website.

Blocks:  mozilla bug 462411

Attachments

taborder-based.diff Download (3.0 KB) - added by davidb 22 months ago.
the taborder based fix (removed unecessary aria-hidden)
3553.diff Download (3.7 KB) - added by davidb 22 months ago.
uses arrows. one header in tab order. removes header links from tab order needs review
3553.2.diff Download (3.8 KB) - added by davidb 22 months ago.
this one doesn't auto-expand when arrowing.
3553.3.diff Download (4.1 KB) - added by davidb 22 months ago.
makes sure at least one header in tab order, improved tabindex modification
3553toCommit.diff Download (4.1 KB) - added by davidb 22 months ago.
patch just committed to trunk. tested in FF2/3, Safari, IE8

Change History

Changed 22 months ago by davidb

  • keywords michelled added

When I get a chance, I'll post a patch that follows the  http://dev.aol.com/dhtml_style_guide; unless we want to keep the 'every header is in the tab order' keyboard interaction. It will be a bit more work to follow the styleguide, and currently there is an agenda item to review an accordion proposal from James Craig, which I believe is more akin to the current patch.

Changed 22 months ago by davidb

the taborder based fix (removed unecessary aria-hidden)

Changed 22 months ago by davidb

uses arrows. one header in tab order. removes header links from tab order needs review

Changed 22 months ago by davidb

this one doesn't auto-expand when arrowing.

Changed 22 months ago by davidb

makes sure at least one header in tab order, improved tabindex modification

Changed 22 months ago by davidb

  • cc michelled added
  • keywords michelled removed

Changed 22 months ago by davidb

patch just committed to trunk. tested in FF2/3, Safari, IE8

Changed 20 months ago by scott.gonzalez

  • milestone changed from TBD to 1.next
Note: See TracTickets for help on using tickets.