-
Notifications
You must be signed in to change notification settings - Fork 842
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
More Icon Additions for Upcoming Lens Features #4833
Conversation
Preview documentation changes for this PR: https://eui.elastic.co/pr_4833/ |
I'm curious why not just use the |
We've received some user feedback in recent Lens testing that the |
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.
Makes sense. I would make sure to communicate this out to all the rest of the designers and make sure add them to the Figma library.
# Conflicts: # CHANGELOG.md
Preview documentation changes for this PR: https://eui.elastic.co/pr_4833/ |
Summary
Added some more new icons to EUI to support the needs of upcoming Lens features (thresholds, custom formula). This PR includes the following icons:
documentation
layers
Checklist
- [ ] Checked in mobile- [ ] Props have proper autodocs and **[playground toggles](https://github.com/elastic/eui/blob/master/wiki/documentation-guidelines.md#adding-playground-toggles)**- [ ] Checked Code Sandbox works for the any docs examples- [ ] Checked for breaking changes and labeled appropriately- [ ] Checked for accessibility including keyboard-only and screenreader modes