-
Notifications
You must be signed in to change notification settings - Fork 378
Fixes demo to show single and multiple expand behavior #1860
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
Conversation
|
PatternFly-React preview: https://1860-pr-patternfly-react-patternfly.surge.sh |
Codecov Report
@@ Coverage Diff @@
## master #1860 +/- ##
==========================================
- Coverage 82.59% 82.59% -0.01%
==========================================
Files 621 622 +1
Lines 6844 6849 +5
Branches 93 93
==========================================
+ Hits 5653 5657 +4
- Misses 1151 1152 +1
Partials 40 40
Continue to review full report at Codecov.
|
mcarrano
left a comment
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.
This now behaves as I expect. Thanks @dlabaj !
tlabaj
left a comment
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.
LGTM
jessiehuff
left a comment
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.
LGTM!
tlabaj
left a comment
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.
LGTM
|
@dlabaj can you get rid of the span that wraps the svg? It's not necessary. |
|
@mcoker sure thing. Doing it now. |
2ff69e7
tlabaj
left a comment
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.
LGTM

Fixes the behavior of the accordion for #1387 and makes accordion available to consumers.
Additional issues:
Please merge this PR first #1852 (review) I'll rebase after to get these in.