8000 fix(data-list): add spacing to compact data-list buttons by sg00dwin · Pull Request #7189 · patternfly/patternfly · GitHub
[go: up one dir, main page]

Skip to content

fix(data-list): add spacing to compact data-list buttons#7189

Merged
mcoker merged 1 commit intopatternfly:mainfrom
sg00dwin:7105-datalist
Nov 19, 2024
Merged

fix(data-list): add spacing to compact data-list buttons#7189
mcoker merged 1 commit intopatternfly:mainfrom
sg00dwin:7105-datalist

Conversation

@sg00dwin
Copy link
Contributor
@sg00dwin sg00dwin commented Nov 1, 2024

fixes #7105

  • Update DataList compact examples to use small buttons
  • Add top and bottom padding to small buttons

Backstop tests
https://drive.google.com/drive/folders/1aKh3N42bPbMjsfkTTbGOrQr8zUO_YHYn?usp=sharing

Before
Screenshot 2024-11-01 at 6 13 52 PM

After
Screenshot 2024-11-01 at 6 13 32 PM

Before
Screenshot 2024-11-01 at 6 51 25 PM

After
Screenshot 2024-11-01 at 6 50 0
8000
1 PM

@patternfly-build
Copy link
Collaborator
patternfly-build commented Nov 1, 2024

Copy link
Contributor
@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

LGTM!! Can you also update the actions in this example to use the small variation? https://patternfly-pr-7189.surge.sh/components/data-list/html/expandable-compact/

One other thing, I love the hbs data-list--IsCompact parameter - totally optional or could be a follow up, but it would be great if that could set .pf-m-compact as well. Otherwise it's a little confusing that a .pf-m-compact data-list also needs data-list--IsCompact to attempt to resize its buttons. As far as I can tell there are only 4 compact data lists in our docs that would need to be updated.

{{> @partial-block}}
{{else}}
{{> menu-toggle
menu-toggle--IsSmall=data-list--IsCompact
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@sg00dwin sg00dwin force-pushed the 7105-datalist branch 2 times, most recently from 355dc77 to c36096a Compare November 14, 2024 20:39
Copy link
Member
@srambach srambach left a comment

Choose a reason for hiding this comment

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

Looks great! 👍🏻

@mcoker
Copy link
Contributor
mcoker commented Nov 18, 2024

@sg00dwin can you rebase this branch to see if it resolves the build error?

Copy link
Contributor
@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

🔥

@mcoker mcoker merged commit a785ab6 into patternfly:main Nov 19, 2024
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.1.0-prerelease.5 🎉

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.

Datalist - There is not spacing above the action buttons when datalist is compact

4 participants

0