-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update some icons to address aliasing on high-res screens #49381
Conversation
Size Change: +902 B (0%) Total Size: 1.34 MB
ℹ️ View Unchanged
|
Really stellar work. Are you also updating and publishing the Figma library as part of this? The gallery code looks slightly more expansive. Can you maybe run that through https://svgomg.net/? |
I did with one of the Figma plugins, ran it again. I can't seem to get it without
Yes! I have them ready, was waiting for approval first. I'll move these standouts into the actual icon components when we're merged here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an awesome attention to detail, thank you. Let's get this in.
Edit: just one small fix first.
Flaky tests detected in 0032169. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4568100299
|
Part of ##49311, focusing on common blocks/UI icons first. Updates a few icons to remove the aliasing that occurs on high-resolution screens.
Why?
We want our icons to look crisp and present well.
How?
Updating icons within the icons package. This does not change the spirit of the icons, just adjusts the icon paths a little to not use
0.25
X and Y values.Testing Instructions
Check the icons in the following locations:
Screenshots or screencast