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

Accessibility - Collapse accordion example - wrong aria attribute multiselectable #21635

Closed
Lausselloic opened this issue Jan 10, 2017 · 3 comments

Comments

@Lausselloic
Copy link
Contributor

Hello,

The accordion sample based on collapse plugin use the aria attribute multiselectable set to true in the documentation. But this accordion is not a multiple open at a time, each new panel opening, close the previous one. From my point of view, this attribute need to be remove until a multi open at a time option will be propose.

http://v4-alpha.getbootstrap.com/components/collapse/#accordion-example

What's your opinion @patrickhlauke ?

@patrickhlauke
Copy link
Member

agree, good catch

@jkstermitz
Copy link

jkstermitz commented Jan 26, 2017

Edited.....seems I found the other issue indicating Card is needed for accordion to work. My bad.....ignore the following.

Help me understand this please? I'm trying to use the accordion example to implement an expanding vertical nav, with two menus. I WANT one menu to collapse when I open the other (and vice versa), and have set it up identical to the example....but of course it does not work as the sample code has "true" set.

I removed the aria-multiselectable attribute and also tried leaving it in but setting it to "false," but neither seem to affect the behavior (both menus are always independently expandable).

So, is "multi-selectable="false" supposed to work at all? How is it that the example functions properly? Inspecting the code doesn't provide a clue. It seems the data-parent="#accordion" is what is not working.....

@mportuga
Copy link

mportuga commented Aug 6, 2018

@jkstermitz That is just for accessibility. That attribute tell a blind user what behavior to expect.

http://www.maxability.co.in/2015/06/aria-multiselectable-property/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants