Skip to content

docs(menu, select, app launcher, dropdown): Updates images for v6.#4643

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

docs(menu, select, app launcher, dropdown): Updates images for v6.#4643
andrew-ronaldson merged 6 commits into
patternfly:mainfrom
bekah-stephens:iss4555

Conversation

@bekah-stephens
Copy link
Copy Markdown
Contributor

Closes #4555

@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Jun 3, 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.

just a couple things, also opened a docs followup: #4647

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is a little nit picky, but it's a bit difficult for me to see what's going on in this image due to the size (especially the group header and the truncation). Could you try scooting the 3 examples closer together to see if that helps when they're scaled?

Might just be me right now, if we can't do much with it that's fine

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I tweaked the spacing a little bit, but not sure if that helped?

@andrew-ronaldson
Copy link
Copy Markdown
Contributor

For the split button image do we need the toggle group and input groups in the image?

Screenshot 2025-06-12 at 7 49 15 AM

Under select it looks like the checkboxes in the Figma file didn't show in the PR.
Screenshot 2025-06-12 at 7 49 25 AM
Screenshot 2025-06-12 at 7 49 32 AM

@bekah-stephens bekah-stephens requested a review from edonehoo July 2, 2025 15:03
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.

tyty!

@andrew-ronaldson andrew-ronaldson merged commit 2a55a9d into patternfly:main Jul 2, 2025
4 checks passed
nicolethoen pushed a commit to nicolethoen/patternfly-org that referenced this pull request Aug 1, 2025
…atternfly#4643)

* docs(menu, select, app launcher, dropdown): Updates images for v6.

* Updating images.

* Updating image issues.

* Adding badge count to menu

* Fixing menu text

---------

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: Menu, Select, dropdown & Menu toggle

4 participants