-
Notifications
You must be signed in to change notification settings - Fork 709
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
Give interface SVGs the business #1745
Conversation
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.
A couple of changes to be made, but otherwise, this is fantastic. Thank you!
@tigt we have a bit of a conflict on the logos (as they are new), then I think we'll be good to go |
Should I omit them from this PR to make it easier to merge? |
I think that’s a great idea |
This is mostly SVGOMG, but also I put some extra stink on it to make a strong impression and and rewrote a lot of them to be even smaller. GitHub’s rich diff viewer should come in very handy here.
Specific notes:
For
icon-webvitals-testing
:letter-spacing
was previously used to make the word’s bounding box centered, buttextLength
is better-suited since it will Do The Right Thing if the font metrics ofsans-serif
are differentFor
icon-advanced-testing
: I was able to remove thecrispEdges
since the overhauled dimensions all became integers, but if that still doesn’t render correctly on some browsers it can be restoredThis also fixes a very subtle visual bug in the data URIs for the dropdown arrow: