-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Icon Updates] Remove masks on svgs for glyphs a-m #187
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.
LGTM!
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.
🥇
</g> | ||
</mask> | ||
<g mask="url(#mask0)"> | ||
<rect width="48" height="48" fill="currentColor"/> | ||
</g> | ||
<defs> | ||
<clipPath id="clip0"> | ||
<rect x="8" y="10" width="32" height="29" fill="white"/> | ||
</clipPath> | ||
</defs> |
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.
i'm sure you're taking notes on these, but we'll prob want to double check the more complicated ones like this.
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.
Checking as I was updating, they seem unaffected by removing these extra lines, but will definitely check them in the other repos as well.
Discovered that using the masks from the exported Figma SVGs was causing some issues in some use cases with fills (where the fill would cover the whole square of the SVG instead of just the path).
This PR removes the masks on the glyphs A-M