-
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] Add / update 3/3 icons from Figma #182
Conversation
@@ -37,15 +36,6 @@ class OptionButtonExample extends React.Component { | |||
type="primary" | |||
icon={<PlusIcon />} | |||
/> | |||
<OptionButton |
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.
was thi removed in other PR?
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.
sry I refer to the EqualsIcon
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.
Just aesthetic. The new EqualsIcon
is now wrapped in a circle, and so it looked like this:
It's on my radar to check how it's used in the main repos and bring in a designer to decide if any changes are needed, but for this it was just being used as an example for the button, and it it was looking strange next to the plus and minus icons.
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.
yeah that looks weird.
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.
👍
Asana: https://app.asana.com/0/0/1158576271189801/f
Glyph Updates sheet: https://docs.google.com/spreadsheets/d/1FNiVhL3eV_Vc69JZycn4QAz63HOSXBb4RZAsHSc9XdY/edit?usp=sharing
In this PR:
Note: Some of the icons have substantial design differences between the old and the new, and for those I will be checking both Gatsby and PocketDerm repos once this is published and the package is ready to be updated. There are also some icons which have been renamed, so the old names will be removed in a later PR once those repos have been updated.