Skip to content

docs(table): Updates images for v6.#4676

Merged
andrew-ronaldson merged 2 commits into
patternfly:mainfrom
bekah-stephens:iss4569
Jul 14, 2025
Merged

docs(table): Updates images for v6.#4676
andrew-ronaldson merged 2 commits into
patternfly:mainfrom
bekah-stephens:iss4569

Conversation

@bekah-stephens
Copy link
Copy Markdown
Contributor

Closes #4569

@bekah-stephens
Copy link
Copy Markdown
Contributor Author

@edonehoo @andrew-ronaldson I left the spacers off of the screenshots in the Spacing section since they didn't quite line up. let me know if we need to add them back in.

@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator

@edonehoo edonehoo left a comment

Choose a reason for hiding this comment

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

having spacers in the compact/default spacing images could be helpful to more clearly highlight the style differences between the two, but I can also kind of already notice the differences without them, so I'll leave that up to y'all

@bekah-stephens
Copy link
Copy Markdown
Contributor Author

having spacers in the compact/default spacing images could be helpful to more clearly highlight the style differences between the two, but I can also kind of already notice the differences without them, so I'll leave that up to y'all

I have them hidden in Figma and chose not to show them because I couldn't get the spacers to line up with the components. Happy to show them, I just need to figure out which tokens they're supposed to be using.

@andrew-ronaldson andrew-ronaldson merged commit 17e9690 into patternfly:main Jul 14, 2025
4 checks passed
nicolethoen pushed a commit to nicolethoen/patternfly-org that referenced this pull request Aug 1, 2025
Co-authored-by: Andrew Ronaldson <aronalds@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Guideline updates: Table

4 participants