Skip to content

[8.x] Upgrade EUI to 101.3.0-classic.0 #217395

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
weronikaolejniczak merged 4 commits into from
Apr 10, 2025
Merged

[8.x] Upgrade EUI to 101.3.0-classic.0 #217395

weronikaolejniczak merged 4 commits into from
Apr 10, 2025

Conversation

@tkajtoch
Copy link
Member

@tkajtoch tkajtoch commented Apr 7, 2025 •
edited by weronikaolejniczak
Loading

Note

This PR is a manual backport of the already merged #217491

101.0.1 ⏩ 101.3.0-classic.0

Questions? Please see our Kibana upgrade FAQ.

Package updates

@elastic/eui

v101.3.0

  • Updated 78 existing and added two new glyphs (code and checkCircle) for EuiIcon (#8530)
  • Changed gutterSize to m between right side items on EuiPageHeader (#8529)

Bug fixes

  • Fixed a visual bug on disabled EuiButton in high contrast mode where wrong text colors were applied (#8550)

v101.2.0

  • Added showToolTip prop on EuiColorPickerSwatch (#8512)

Bug fixes

  • Fixed a visual issue of overlapping borders for layered EuiPanels (#8519)
  • Fixes wrong colorMode styling for the search in EuiHeader with theme="dark" (#8496)

Accessibility

  • Improved the accessibility of EuiColorPicker by adding color label tooltips on hover and focus for color swatches (#8512)
  • Added disableScreenReaderOutput prop on EuiToolTip to manually control if the tooltip content should be read when focusing the trigger. This prevents duplicate screen reader output when the tooltip content and aria-label on the trigger element have the same text content. (#8508)
  • Improves text color contrast for EuiButton with color="warning" in high contrast mode (#8496)
  • Improves contrast and visible distinction of the following components in high contrast mode: (#8496)
    • EuiCode
    • EuiBadge
    • EuiBetaBadge
    • EuiNotificationBadge

Dependency updates

  • Updated prismjs to v1.30.0 (#8506)

v101.1.0

  • Updates EuiTableRow styles to check support for :has(+) selector (#8498)

@tkajtoch tkajtoch self-assigned this Apr 7, 2025
@tkajtoch tkajtoch added backport:skip This PR does not require backporting EUI release_note:skip Skip the PR/issue when compiling release notes backport This PR is a backport of another PR labels Apr 7, 2025
@weronikaolejniczak weronikaolejniczak changed the title [8.x] Upgrade EUI to 101.2.0-classic.0 [8.x] Upgrade EUI to 101.3.0-classic.0 Apr 8, 2025
@weronikaolejniczak weronikaolejniczak force-pushed the feat/eui-101.2.0-classic.0 branch 2 times, most recently from c4c4f1a to 0ca0455 Compare April 9, 2025 09:22
@weronikaolejniczak weronikaolejniczak marked this pull request as ready for review April 9, 2025 11:06
@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 9, 2025

Pinging @elastic/eui-team (EUI)

@weronikaolejniczak weronikaolejniczak force-pushed the feat/eui-101.2.0-classic.0 branch from 0ca0455 to 644cf74 Compare April 9, 2025 11:17
@botelastic botelastic bot added Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. labels Apr 9, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 9, 2025

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 9, 2025

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@weronikaolejniczak weronikaolejniczak requested review from a team April 9, 2025 11:20
@weronikaolejniczak weronikaolejniczak requested review from a team and rylnd and removed request for a team April 9, 2025 11:21
consulthys
consulthys approved these changes Apr 9, 2025
View reviewed changes
Copy link
Contributor

@consulthys consulthys left a comment

Choose a reason for hiding this comment

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

LGT Stack Monitoring

MiriamAparicio
MiriamAparicio approved these changes Apr 9, 2025
View reviewed changes
Copy link
Contributor

@MiriamAparicio MiriamAparicio left a comment

Choose a reason for hiding this comment

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

Changes for obs-ux-infra-services LGTM

peteharverson
peteharverson approved these changes Apr 9, 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.

Edit to ML snapshot test LGTM

@weronikaolejniczak weronikaolejniczak requested a review from a team April 9, 2025 12:07
akowalska622
akowalska622 approved these changes Apr 9, 2025
View reviewed changes
Copy link
Contributor

@akowalska622 akowalska622 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.

Data Discovery changes LGTM

@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 9, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #2 / useGetCases shows a toast error message when an error occurs in the response

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.1KB 478.3KB +114.0B
dashboard 614.8KB 614.9KB +114.0B
infra 1.6MB 1.6MB +113.0B
kubernetesSecurity 263.1KB 263.2KB +114.0B
maps 3.1MB 3.1MB +114.0B
ml 5.4MB 5.4MB +113.0B
visTypeVega 2.1MB 2.1MB +113.0B
total +795.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 +1.3KB

History

cc @weronikaolejniczak

yansavitski
yansavitski approved these changes Apr 9, 2025
View reviewed changes
Copy link
Contributor

@yansavitski yansavitski left a comment

Choose a reason for hiding this comment

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

LGTM

rylnd
rylnd approved these changes Apr 9, 2025
View reviewed changes
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 changes LGTM. None of our owned files were changed, but some snapshots related to tooltips were updated so I clicked through the relevant areas on my local machine.

@weronikaolejniczak weronikaolejniczak merged commit 58c9132 into elastic:8.x Apr 10, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rylnd rylnd rylnd approved these changes

@consulthys consulthys consulthys approved these changes

@markov00 markov00 markov00 approved these changes

@mbondyra mbondyra mbondyra approved these changes

@delanni delanni delanni approved these changes

@peteharverson peteharverson peteharverson approved these changes

@Dosant Dosant Dosant approved these changes

@jloleysens jloleysens jloleysens approved these changes

@yansavitski yansavitski yansavitski approved these changes

@MiriamAparicio MiriamAparicio MiriamAparicio approved these changes

@agusruidiazgd agusruidiazgd agusruidiazgd approved these changes

@akowalska622 akowalska622 akowalska622 approved these changes

@benakansara benakansara benakansara approved these changes

@elena-shostak elena-shostak elena-shostak approved these changes

@kibanamachine kibanamachine Awaiting requested review from kibanamachine

Assignees

@weronikaolejniczak weronikaolejniczak

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 Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.