Skip to content

fix(dual-list-selector): fix overflow of items - #7861

Merged
mcoker merged 2 commits into
patternfly:mainfrom
srambach:7854-dual-list-draggable-overflow-bug
Sep 23, 2025
Merged

mcoker merged 2 commits into
patternfly:mainfrom
srambach:7854-dual-list-draggable-overflow-bug

Conversation

@srambach

Copy link
Copy Markdown
Member

Fixes #7854

@patternfly-build

patternfly-build commented Sep 23, 2025 •

Copy link
Copy Markdown
Collaborator

@srambach srambach assigned srambach and unassigned srambach Sep 23, 2025
@mcoker
mcoker self-requested a review September 23, 2025 15:26

@mcoker mcoker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment. Curious what you think about that. The PR is fine as-is if you'd rather go with what you have now.

.#{$dual-list-selector}__item,
.#{$dual-list-selector}__item-main {
flex-basis: 100%;
flex-basis: 100%; // TODO

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wdyt about replacing this with flex-grow: 1? Seems like that might be more predictable than using a percentage based size just to ensure the element takes up the available space by default.

Also did you mean to leave the TODO comment?

@mcoker mcoker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mcoker
mcoker merged commit 4b05d73 into patternfly:main Sep 23, 2025
4 checks passed
@patternfly-build

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 6.3.0-prerelease.75 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug - Dual list selector - long text overflows with draggable items

3 participants