Skip to content

Update EUI to v107.0.1 #239744

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.

Sign up for GitHub

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

Jump to bottom
Merged
acstll merged 8 commits into from
Oct 23, 2025
Merged

Update EUI to v107.0.1 #239744

acstll merged 8 commits into from
Oct 23, 2025

Conversation

@acstll
Copy link
Contributor

@acstll acstll commented Oct 20, 2025 •
edited
Loading

  • @elastic/eui: v107.0.0 ⏩ v107.0.1
  • @elastic/eslint-plugin-eui: v2.4.0 ⏩ v2.5.0

Questions? Please see our Kibana upgrade FAQ.

Changes

Related to elastic/eui#9100

  • Updated snapshot f90ae29
  • Added missing aria-label for EuiBetaBadge
    • [Reporting] 6d10edd
    • [ML] ce58009
      • Added new translation key, ran node scripts/i18n_check — is there anything else that should be done in this regard?

Package updates

@elastic/eui v107.0.1

Bug fixes

  • Fixed prismjs theme in EuiCodeBlock to improve highlighting for the yaml language (#9089)
  • Fixed a visual bug on EuiTable where the border for rows in dark mode wasn't applied correctly (#9115)

Dependency updates

  • Updated @elastic/prismjs-esql to v1.1.2 (#9102)

Accessibility

  • Fixed incorrect role attribute on EuiIcon and EuiBetaBadge (#9100)
  • Make EuiBasicTable respect user's reduced motion setting by not animating when in loading state. (#9095)

@elastic/eslint-plugin-eui v2.5.0

  • Added new accessible-interactive-element rule. (#9093)
  • Added new tooltip-focusable-anchor rule. (#9051)
  • Excluded EuiButtonEmpty from the no-unnamed-interactive-element rule. (#9046)

Bug fixes

  • Fixed no-css-color rule to allow CSS keywords like currentcolor, transparent, and inherit (#9092)

@acstll acstll self-assigned this Oct 20, 2025
@acstll acstll added release_note:skip Skip the PR/issue when compiling release notes EUI backport:skip This PR does not require backporting ci:cloud-deploy Create or update a Cloud deployment labels Oct 20, 2025
@acstll acstll changed the title Update EUI to 107.0.1 Update EUI to v107.0.1 Oct 20, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Oct 20, 2025

Cloud deployment initiated, see credentials at: https://buildkite.com/elastic/kibana-deploy-cloud-from-pr/builds/493

@acstll
Copy link
Contributor Author

acstll commented Oct 21, 2025

@elasticmachine merge upstream

@alexwizp alexwizp mentioned this pull request Oct 21, 2025
Closed
acstll added 3 commits October 22, 2025 15:35
@acstll
[Security Solution][Tests] Update snapshot
f90ae29
@acstll
[ReponseOps][Reporting] Add required aria-label to Technical Preview …
6d10edd 
…badge

fixes an AXE aria-command-name violation
@acstll
[ML] Add required aria-label to Technical Preview badge
ce58009 
- fixes an AXE aria-command-name violation
- translations are missing
@acstll
Copy link
Contributor Author

acstll commented Oct 22, 2025

@elasticmachine merge upstream

@acstll
Copy link
Contributor Author

acstll commented Oct 23, 2025

@elasticmachine merge upstream

@acstll
Copy link
Contributor Author

acstll commented Oct 23, 2025 •
edited
Loading

I cannot be certain but the failing FTRs look flaky and unrelated to the changes in this PR.

In all cases it's like there's a delta of 2 in an array resulting of an api call (?):

I'm opening for review. Any hints more than welcome.

@acstll acstll marked this pull request as ready for review October 23, 2025 11:17
@acstll acstll requested review from a team as code owners October 23, 2025 11:17
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 23, 2025

Pinging @elastic/eui-team (EUI)

adcoelho
adcoelho approved these changes Oct 23, 2025
View reviewed changes
Copy link
Contributor

@adcoelho adcoelho left a comment

Choose a reason for hiding this comment

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

response-ops OK

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 23, 2025 •
edited
Loading

💚 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
ml 5.4MB 5.4MB +268.0B
reporting 184.3KB 184.4KB +19.0B
total +287.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 6.5MB 6.5MB +815.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 698 696 -2
streamsApp 19 17 -2
total -4

Total ESLint disabled count

id before after diff
securitySolution 801 799 -2
streamsApp 30 28 -2
total -4

History

cc @acstll

peteharverson
peteharverson approved these changes Oct 23, 2025
View reviewed changes
Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

ML changes with the internationalized aria-labels for the EuiBetaBadge LGTM

PhilippeOberti
PhilippeOberti approved these changes Oct 23, 2025
View reviewed changes
Copy link
Contributor

@PhilippeOberti PhilippeOberti left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM for the @elastic/security-threat-hunting-investigations team

awahab07
awahab07 approved these changes Oct 23, 2025
View reviewed changes
Copy link
Contributor

@awahab07 awahab07 left a comment

Choose a reason for hiding this comment

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

obs-ux-logs changes LGTM! Just two comment lines are removed.

@acstll acstll merged commit ee293d6 into elastic:main Oct 23, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adcoelho adcoelho adcoelho approved these changes

@awahab07 awahab07 awahab07 approved these changes

@spong spong spong approved these changes

@mistic mistic mistic approved these changes

@peteharverson peteharverson peteharverson approved these changes

@PhilippeOberti PhilippeOberti PhilippeOberti approved these changes

Assignees

@acstll acstll

Labels

backport:skip This PR does not require backporting ci:cloud-deploy Create or update a Cloud deployment EUI release_note:skip Skip the PR/issue when compiling release notes v9.3.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@acstll @kibanamachine @elasticmachine @adcoelho @awahab07 @spong @mistic @peteharverson @PhilippeOberti