search

paypal / bootstrap-accessibility-plugin

Accessibility Plugin for Bootstrap 3 and Bootstrap 3 as SubModule
http://paypal.github.io/bootstrap-accessibility-plugin/demo.html
BSD 3-Clause "New" or "Revised" License
789 stars 189 forks source link

Tabindex on accordions #98

Closed juliemoynat closed 8 years ago

juliemoynat commented 8 years ago

Hello,

Bootstrap accessibility plugin set "tabindex" attribute to "0" for the first accordion panel heading and "-1" to the others so users can't navigate from one panel to the others. Tabindex shouldn't be setting for accordions.

And there is an "aria-selected" on the div.panel-collapse that is added. I don't think it is useful here.

P.S.: Why have you deactivated your bootstrap-accessibility.js on your demo site?

juliemoynat commented 8 years ago

Ok, my bad. We should navigate with tab for the first accordion and arrow keys for the next.