-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[icons] Add TipsAndUpdate icon #29004
Conversation
dcf5227
to
8053fda
Compare
8053fda
to
da2fcd3
Compare
Do you know why it was excluded? |
@eps1lon The comment says the following:
Do you think we should keep excluding it? |
Well we already have problems with the size of the icons package. This PR creates a precedence that we actually don't want to ignore new icons. Up to you what direction you want to go. |
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.
👍
Closes #29000
This PR de-excludes
TipsAndUpdates
icon.code sandbox: https://codesandbox.io/s/naughty-feather-2f9y2?file=/src/App.tsx