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
feat(accordion): add custom class on card level #3563
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.
Hey, 2 things here:
-
Div's don't have disabled attribute, so the markup will be invalid → https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/disabled
-
Tests are missing
We should probably explore the way of adding a custom css class (as it is done, for example for modals and tooltips)
569592d
to
ff05200
Compare
Codecov Report
@@ Coverage Diff @@
## master #3563 +/- ##
==========================================
+ Coverage 91.67% 91.67% +<.01%
==========================================
Files 100 100
Lines 2883 2884 +1
Branches 532 532
==========================================
+ Hits 2643 2644 +1
Misses 183 183
Partials 57 57
Continue to review full report at Codecov.
|
ff05200
to
8aa1564
Compare
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 for me too, thanks!
Enable customization of accordion panel conditionally according to being disabled or not.
Fixes #2262