Skip to content

[8.19] Upgrade EUI to v101.4.0#218823

Merged
mgadewoll merged 3 commits intoelastic:8.19from
mgadewoll:eui/v101.4.0-8.19
Apr 24, 2025
Merged

[8.19] Upgrade EUI to v101.4.0#218823
mgadewoll merged 3 commits intoelastic:8.19from
mgadewoll:eui/v101.4.0-8.19

Conversation

@mgadewoll
Copy link
Contributor

@mgadewoll mgadewoll commented Apr 22, 2025

101.3.0101.4.0

Questions? Please see our Kibana upgrade FAQ.

Important

This PR is a direct sibling to this upgrade PR to main. The difference is that it adds a standalone EUI package with the previous "Amsterdam" theme.
Apart from the theme difference, there are no further changes added.

Changes

This PR only updates tests and snapshots related to changes on EUI side:

  • updated aria-current="true" to aria-current="page"
  • updated icon usage userAvatar to user

Package updates

@elastic/eui

v101.4.0

  • Spread labelProps to the label element in EuiCheckableCard (#8586)
  • Add controls, flask, comment, and readOnly glyphs to EuiIcon (#8580)
  • Refactored EuiExpression, EuiFacetGroup, EuiFacetButton, EuiFilterGroup, EuiHeader, EuiImage and EuiListGroup to memoize their internal Emotion styles (#8565)
  • Updated global border.radius.medium token value for default Borealis theme to 4px (#8563)
  • Updated EuiProvider to build themes including highContrastMode (#8558)

Accessibility

  • Removed the aria-label attribute from the ul element in EuiPagination to avoid duplicate screen reader output (#8597)
  • Set a more specific aria-current="page" on list items in EuiPagination (#8597)
  • Added aria-modal to EuiFlyout with type="overlay" (#8591)

Dependency updates

  • Updated @elastic/prismjs-esql to v1.1.0 (#8587)

@elastic/eui-theme-borealis@0.2.0

  • Updated component tokens to use computed values to ensure correct inheritance from theme overrides (#8558)
  • Added overrides.HCM to euiThemeBorealis to support theme internal overrides (#8558)
  • Updated border.radius.medium token value to 4px (#8563)

@elastic/eui-theme-common@0.2.0

  • Added support for theme overrides as optional part of EuiThemeShape (#8558)
  • Updated getComputed to support high contrast mode overrides defined on overrides.HCM (#8558)

@mgadewoll mgadewoll added backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes EUI backport:skip This PR does not require backporting labels Apr 22, 2025
@mgadewoll mgadewoll requested review from a team April 22, 2025 13:54
@mgadewoll mgadewoll self-assigned this Apr 22, 2025
@mgadewoll mgadewoll requested review from dhurley14, parkiino, paul-tavares and tiansivive and removed request for a team April 22, 2025 13:54
- updates assertions on aria-current value from true to page, updated replaced icon
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
controls 478.2KB 478.2KB +6.0B
dashboard 612.8KB 612.8KB +6.0B
infra 1.6MB 1.6MB +6.0B
kubernetesSecurity 263.2KB 263.2KB +6.0B
maps 3.1MB 3.1MB +6.0B
ml 5.4MB 5.4MB +6.0B
visTypeVega 2.1MB 2.1MB +6.0B
total +42.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-npmDll 5.9MB 5.9MB -183.0B

History

cc @mgadewoll

@mgadewoll mgadewoll marked this pull request as ready for review April 23, 2025 09:30
@elasticmachine
Copy link
Contributor

Pinging @elastic/eui-team (EUI)

Copy link
Contributor

@tomsonpl tomsonpl left a comment

Choose a reason for hiding this comment

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

Defend Workflows LGTM

Copy link
Member

@machadoum machadoum left a comment

Choose a reason for hiding this comment

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

Entity analytics changes LGTM!

Copy link
Contributor

@paul-tavares paul-tavares left a comment

Choose a reason for hiding this comment

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

LGTM - elastic/security-defend-workflows team

Copy link
Contributor

@christineweng christineweng left a comment

Choose a reason for hiding this comment

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

Investigations changes LGTM

Copy link
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

Detection Engine LGTM

@mgadewoll mgadewoll merged commit a2d08ab into elastic:8.19 Apr 24, 2025
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting backport This PR is a backport of another PR EUI release_note:skip Skip the PR/issue when compiling release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.