Skip to content

chore(many): design guidelines assets#4498

Merged
andrew-ronaldson merged 12 commits intopatternfly:mainfrom
andrew-ronaldson:design-guidelines
May 1, 2025
Merged

chore(many): design guidelines assets#4498
andrew-ronaldson merged 12 commits intopatternfly:mainfrom
andrew-ronaldson:design-guidelines

Conversation

@andrew-ronaldson
Copy link
Copy Markdown
Contributor

@andrew-ronaldson andrew-ronaldson commented Mar 12, 2025

Closing:
#4273
#4462
#4463

@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Mar 12, 2025

Copy link
Copy Markdown
Contributor

@bekah-stephens bekah-stephens left a comment

Choose a reason for hiding this comment

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

lgtm! (did a quick pass-through, but didn't look at screenshot specifics)

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.

I called out a few hopefully easy fixes, but lmk if you have thoughts/concerns with any of my suggestions! Ty for talking the alert beast

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.

I checked the figma file and it looks like the token marker annotations were outside of the frame, so they didn't export with this image. I fixed it in figma, so if you re-export and upload then I think you'll be good to go

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 one is also missing annotations!

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.

I know this image is replicating the existing one, but I feel like the current one is longer than it needs to be. Could we try making the image more of a horizontal spread? I made a new frame in figma called "alert-types-erin" (which you can just rename) for what I was thinking, but I'm open to feedback if we want to keep this format

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.

For this one, can you change the field label for the 2 errored inputs to be "Version" and "Region"? Just to align with the link names used in the error

@andrew-ronaldson andrew-ronaldson merged commit 31c2e44 into patternfly:main May 1, 2025
4 checks passed
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.

4 participants