Allow setting multiselectable via options

nico3333fr / jquery-accessible-accordion-aria

jQuery Accessible Accordion System, using ARIA

MIT License

60 stars 19 forks source link

Allow setting multiselectable via options #18

Closed swissspidy closed 7 years ago

swissspidy commented 7 years ago

If you pass multiselectable as an option like any other, its value gets overridden by specificOptions.multiselectable (the data-accordion-multiselectable) attribute.

I think it makes sense to not override it as long as no data attribute is present, e.g. when I use $( '$( '.foobar' ).accordion( { multiselectable: false } );, I expect multiselectable to be false.

nico3333fr commented 7 years ago

You are right, it is fixed now.

Thanks for pointing this out 👍