Skip to content

Conversation

tmorehouse
Copy link
Member

@tmorehouse tmorehouse commented Jun 17, 2017

Adds Accordion support to b-collapse.

Collapses can be turned into an accordion by setting the prop accordion on the collapse components to a common group identifier name. Uses event accordion::toggle emitted on $root to signal other collapses in the group to close

Adds ARIA compliance support to directive v-b-toggle

Addresses issue #513

Includes updated documentation and a test suite

Working example: https://jsfiddle.net/ya15m7dv/

tmorehouse added 27 commits May 15, 2017 21:57
Coerce slide nodelist into an Array
Adds support for accordion by adding a new accordion prop that identifies a group of collapses.

When the collapse in the accordion group is shown, it emits an `accordion::toggle` event on $root to signal other collapses in this accordion group to close.
Adds ARIA support to directive v-b-toggle
@tmorehouse tmorehouse changed the title [collapse] Add Accordion support [collapse] Add Accordion support [WIP] Jun 17, 2017
@tmorehouse tmorehouse changed the title [collapse] Add Accordion support [WIP] [collapse] Add Accordion support and test suite [WIP] Jun 17, 2017
@tmorehouse tmorehouse changed the title [collapse] Add Accordion support and test suite [WIP] [collapse] Add Accordion support, ARIA, and test suite Jun 17, 2017
@tmorehouse tmorehouse requested a review from mosinve June 17, 2017 08:31
@tmorehouse tmorehouse added this to the v0.17.0 milestone Jun 17, 2017
@tmorehouse tmorehouse merged commit f5c094c into bootstrap-vue:master Jun 17, 2017
@kylemh
Copy link
Contributor

kylemh commented Jul 14, 2017

Does this implementation of an Accordion support nested items? Can I create an Accordion within <b-card> components? I tried doing so, but am failing miserably without documentation.

@mosinve
Copy link
Member

mosinve commented Jul 14, 2017

Is this fiddle covers your question?
https://jsfiddle.net/cihkem/n5rgjvfj/3/
You can do even this way https://jsfiddle.net/cihkem/2mLv0xyL/

@kylemh
Copy link
Contributor

kylemh commented Jul 14, 2017

You're the man!

@mosinve
Copy link
Member

mosinve commented Jul 14, 2017

Second example was not saved... Now all ok.

And with next release BV will allow to use v-b-toggle directive on any element at page, even if it hasn't Vue instance.

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

Successfully merging this pull request may close these issues.

3 participants