-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Allow nested structure for accordions #25121
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a unit test
This commit allows nested structures for accordions. Also a part of the documentation about data-children is removed because this functionality didn't work and it's not applicable anymore. Tests with the collapse accordion are also a bit adjusted to the new situation.
…trap into collapse-fix-parent
|
docs/4.0/components/collapse.md
Outdated
@@ -63,7 +63,7 @@ Multiple `<button>` or `<a>` can show and hide an element if they each reference | |||
|
|||
## Accordion example | |||
|
|||
Using the [card]({{ site.baseurl }}/docs/{{ site.docs_version }}/components/card/) component, you can extend the default collapse behavior to create an accordion. | |||
You can extend the default collapse behavior to create an accordion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, this doesn't seem right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @XhmikosR
* dist * Outline active focus (twbs#25145) * Base the outline button :active color on the background, fixing a contrast issue * Only apply focus outline to active when it's focused * Add modal-dialog-centered class to vertically centered modal example. * removes hardcoded value for nav-link padding * Apply of twbs#25068 to input group * dist * docs: Fix overflow issue in IE. (twbs#25142) * Allow nested structure for accordions (twbs#25121) This commit allows nested structures for accordions. Also a part of the documentation about data-children is removed because this functionality didn't work and it's not applicable anymore. Tests with the collapse accordion are also a bit adjusted to the new situation.
Fixes #25119