-
Notifications
You must be signed in to change notification settings - Fork 91
Add groups/dividers to PF4 select #1148
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
A new version (feat) will be released: v3.15.0 [DataDrivenFormsBot] |
Codecov Report
@@ Coverage Diff @@
## master #1148 +/- ##
==========================================
- Coverage 94.91% 94.91% -0.01%
==========================================
Files 208 209 +1
Lines 3542 3576 +34
Branches 1224 1243 +19
==========================================
+ Hits 3362 3394 +32
- Misses 180 182 +2
Continue to review full report at Codecov.
|
- for more generic usage
ad42a37
to
04e124a
Compare
if (item.group) { | ||
return ( | ||
<div className="pf-c-select__menu-group" key={`group-${index}`}> | ||
<div className="pf-c-select__menu-group-title">{item.group}</div> |
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.
@rvsia Looking at the PF4 implementation, they use nested lists in the select options menu. But here we have a flat structure. I think we might hear from QAs that this menu is composed accordingly. I personally don't care that much. Only reason that I can think of why the DOM should be nested is accessibility.
What do you think?
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.
Fixed.
🎉 This PR is included in version 3.15.0 🎉 The release is available on |
I am seeing this now when building, are you seeing the same by any chance? Only started after upgrading (took me awhile to do this).
Edit: |
Strange, force updating the package to
Edit: Edit 2:
That is interesting because i'm using Edit 3: |
Fixes #1145
Description
Adds support for PF4 select group and divider objects.
TODO:
Schema (if applicable)