-
Notifications
You must be signed in to change notification settings - Fork 843
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
Add new bell icon #1447
Add new bell icon #1447
Conversation
When exporting icons from sketch, you'll want to be sure to use SVGO and change the path layer to not be a mask (that will get rid of the extra I did a quick export for you. EUI 2.0 actually makes this a lot easier for you because it doesn't rely on masks. Here's the svg code
|
@cchaos Thanks! I was battling with the SVG export all morning and exporting it with SVGO enabled (Sketch plugin), but also using https://jakearchibald.github.io/svgomg/ to clean it even further. I finally managed to export one that would not have a fill, but I seem to have forgotten about the rest. |
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.
Perfect, LGTM
# Conflicts: # CHANGELOG.md
Summary
Adding a new
bell
icon. We might use this for new alerting/notification features in APM.Checklist
This was checked in mobileThis was checked in IE11Any props added have proper autodocsThis was checked against keyboard-only and screenreader scenariosThis required updates to Framer X components