-
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
Removed svg fills and clips from svg code #24225
Conversation
Size Change: -44 B (0%) Total Size: 1.16 MB
ℹ️ View Unchanged
|
@@ -5,16 +5,8 @@ import { Path, SVG } from '@wordpress/primitives'; | |||
|
|||
const postDate = ( | |||
<SVG fill="none" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"> |
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.
There's one fill left here. Removing it just makes this a tiny bit smaller. No other effect.
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.
There's one blank fill in the SVG element you could remove, but this is good to go. Nice work.
Missed an empty fill
325424e
to
8bab5dc
Compare
Description
The SVG still had fills and clip attributes, so I removed those allowing the icon to receive the proper hover state color.
How has this been tested?
Tested locally.
Screenshots
Types of changes
Non-breaking, SVG changes.
Checklist: