Skip to content

fix: update 60 icons with revised glyphs from EUI batch update (Jan 2026)#3396

Closed
florent-leborgne wants to merge 1 commit into
elastic:mainfrom
florent-leborgne:sync/eui-icons-jan-2026
Closed

fix: update 60 icons with revised glyphs from EUI batch update (Jan 2026)#3396
florent-leborgne wants to merge 1 commit into
elastic:mainfrom
florent-leborgne:sync/eui-icons-jan-2026

Conversation

@florent-leborgne
Copy link
Copy Markdown
Member

Summary

The previous icon sync PR (#3378) only added missing icons but did not update existing ones that changed visually. The EUI Glyph batch updates commit (Jan 9, 2026) revised the artwork for 60 icons without renaming them, so they slipped through the filename-based sync.

Notable icons updated: brush, pencil, share, download, boxes_vertical, calendar, code, database, documentation, error, full_screen, menu, minus, move, plus, question, and 44 more.

What changed

  • 60 SVG files updated to match current EUI glyph designs
  • No renames, no aliases changes — purely visual/content updates

Related: #3378

Made with Cursor

The initial sync only added missing icons but did not update existing
ones that changed visually. The EUI "Glyph batch updates" commit
(c1ef898b, Jan 9 2026) revised the artwork for 60 icons including
brush, pencil, share, download, boxes_vertical, and others.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026

Review Change Stack

Caution

Review failed

The head commit changed during the review from 8822d2e to 1fddee7.

✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@florent-leborgne
Copy link
Copy Markdown
Member Author

Closing in favour of a new PR from origin branch to fix CI permissions issues.

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.

1 participant