Skip to content

Conversation

tkajtoch
Copy link
Member

@tkajtoch tkajtoch commented Sep 12, 2025

Summary

This PR adds a new workflowsApp icon to EuiIcon.

Why are we making this change?

The new workflows application icon is needed soon, and a request to add it was made internally.

Screenshots

Screenshot 2025-09-12 at 18 54 34

Impact to users

This PR adds a single new icon. No impact to end users is expected.

QA

  • Confirm the icon is rendered properly in EuiIcon documentation page

General checklist

  • Browser QA
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox
  • Code quality checklist
  • Release checklist
    • A changelog entry exists and is marked appropriately.

@tkajtoch tkajtoch self-assigned this Sep 12, 2025
@tkajtoch tkajtoch added the icons label Sep 12, 2025
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @tkajtoch

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @tkajtoch

Copy link

@talboren talboren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 🤩

@tkajtoch tkajtoch marked this pull request as ready for review September 15, 2025 11:00
@tkajtoch tkajtoch requested a review from a team as a code owner September 15, 2025 11:00
@acstll acstll self-requested a review September 15, 2025 11:12
Copy link
Contributor

@acstll acstll left a 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?

Screenshot 2025-09-15 at 13 15 17

@tkajtoch
Copy link
Member Author

@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

Copy link
Contributor

@acstll acstll left a 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

@talboren
Copy link

@acstll @tkajtoch quick question though: we're also using this icon in other places like in rules actions:

CleanShot 2025-09-16 at 09 58 52 CleanShot 2025-09-16 at 09 58 46

will we be able to use that simply with EuiIcon too?

@acstll
Copy link
Contributor

acstll commented Sep 16, 2025

quick question though: we're also using this icon in other places like in rules actions:

will we be able to use that simply with EuiIcon too?

yep, that's the expected usage 👍

<EuiIcon type="workflowsApp" />

@tkajtoch tkajtoch merged commit c2c7414 into elastic:main Sep 16, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants