Skip to content

fix(data-list): adjust padding to align drag icon with text#7564

Merged
mcoker merged 1 commit into
patternfly:mainfrom
sg00dwin:7543-bug-data-list-draggable-button-size
Jun 18, 2025
Merged

fix(data-list): adjust padding to align drag icon with text#7564
mcoker merged 1 commit into
patternfly:mainfrom
sg00dwin:7543-bug-data-list-draggable-button-size

Conversation

@sg00dwin

@sg00dwin sg00dwin commented Jun 9, 2025

Copy link
Copy Markdown
Contributor

@patternfly-build

patternfly-build commented Jun 9, 2025

Copy link
Copy Markdown
Collaborator

@srambach srambach left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks fine to me. @mcoker 's comment in the issue sounds like it should be a .pf-m-small button, but it looks like draggable buttons are styled specifically. So unless there's something I'm missing, this looks good!

@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 ab28238 into patternfly:main Jun 18, 2025
4 of 5 checks passed
@patternfly-build

Copy link
Copy Markdown
Collaborator

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

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.

4 participants