-
Notifications
You must be signed in to change notification settings - Fork 858
Add workflowsApp
icon
#9011
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 workflowsApp
icon
#9011
Conversation
💚 Build Succeededcc @tkajtoch |
💚 Build Succeeded
cc @tkajtoch |
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.
All icons under Apps have 2 colors, but this new one does not; is that intended?

@acstll I believe that's expected. The workflows app will only be used in the nav menu where these icons are displayed using a single static color. If that ever changes we can update the icon |
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.
🟢 Looks good to merge
yep, that's the expected usage 👍 <EuiIcon type="workflowsApp" /> |
Summary
This PR adds a new
workflowsApp
icon toEuiIcon
.Why are we making this change?
The new workflows application icon is needed soon, and a request to add it was made internally.
Screenshots
Impact to users
This PR adds a single new icon. No impact to end users is expected.
QA
General checklist