-
Notifications
You must be signed in to change notification settings - Fork 40
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 the select and multiselect filter controls to scroll content #1094
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1094 +/- ##
==========================================
- Coverage 44.11% 44.10% -0.01%
==========================================
Files 177 177
Lines 4477 4487 +10
Branches 997 1001 +4
==========================================
+ Hits 1975 1979 +4
- Misses 2491 2497 +6
Partials 11 11
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
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.
Looks great.
Just waiting to settle PF v5 migration before merging.
16aaebd
to
9a5178e
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.
Also LGTM, but yeah I think it would be easier to rebase this on #1078 than vice versa
9a5178e
to
adf271d
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!
b2da3bb
to
9c97e07
Compare
rebased and updated to work with PF5 |
The PF5 version of the Select component is built on the Menu component, so the Since we're still using the deprecated/PF4 component, the CSS overrides are still needed. |
Add param `isScrollable`, and released css, to allow the patternfly `Select` components at the core of `SelectFilterControl` and `MultiselectFilterControl` to scroll its content as needed. This is a replication of the patternfly Menu `isScrollable` prop[1]. Summary of changes: - Add `isScrollable` (with a default value of `false`) to `SelectFilterControl` and `MultiselectFilterControl` - Add `select-overrides.css` to implement the scrolling for the components - Minor tweaks to the filter select controls to mirror code formatting as they are very similar components Resolves: https://issues.redhat.com/browse/MTA-485 [1]: https://www.patternfly.org/v4/components/menu#scrollable-menus Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
9c97e07
to
d10c149
Compare
Add param `isScrollable`, and released css, to allow the patternfly `Select` components at the core of `SelectFilterControl` and `MultiselectFilterControl` to scroll its content as needed. This is a replication of the patternfly Menu `isScrollable` prop[1]. Summary of changes: - Add `isScrollable` (with a default value of `false`) to `SelectFilterControl` and `MultiselectFilterControl` - Add `select-overrides.css` to implement the scrolling for the components - Minor tweaks to the filter select controls to mirror code formatting as they are very similar components Backport of konveyor#1094 [1]: https://www.patternfly.org/v4/components/menu#scrollable-menus Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
…rols to scroll content (#1113) Add param `isScrollable`, and released css, to allow the patternfly `Select` components at the core of `SelectFilterControl` and `MultiselectFilterControl` to scroll its content as needed. This is a replication of the patternfly Menu `isScrollable` prop[1]. Summary of changes: - Add `isScrollable` (with a default value of `false`) to `SelectFilterControl` and `MultiselectFilterControl` - Add `select-overrides.css` to implement the scrolling for the components - Minor tweaks to the filter select controls to mirror code formatting as they are very similar components Backport of #1094 [1]: https://www.patternfly.org/v4/components/menu#scrollable-menus Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
Add param
isScrollable
, and released css, to allow the patternflySelect
components at the core ofSelectFilterControl
andMultiselectFilterControl
to scroll its content as needed. This is a replication of the patternfly MenuisScrollable
prop[1].Summary of changes:
Add
isScrollable
(with a default value offalse
) toSelectFilterControl
andMultiselectFilterControl
Add
select-overrides.css
to implement the scrolling for the componentsMinor tweaks to the filter select controls to mirror code formatting as they are very similar components
Resolves: https://issues.redhat.com/browse/MTA-485
[1]: https://www.patternfly.org/v4/components/menu#scrollable-menus