Skip to content
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

Upgrade EUI to v98.2.1 #204482

Merged
merged 10 commits into from
Dec 19, 2024
Merged

Upgrade EUI to v98.2.1 #204482

merged 10 commits into from
Dec 19, 2024

Conversation

mgadewoll
Copy link
Contributor

@mgadewoll mgadewoll commented Dec 16, 2024 •
edited
Loading

v98.1.0-borealis.0⏩v98.2.1-borealis.2

Questions? Please see our Kibana upgrade FAQ.

@elastic/eui

v98.2.1

  • Updated the EUI theme color values to use a full 6 char hex code format (#8244)

v98.2.0

  • Added two new icons: contrast and contrastHigh (#8216)
  • Updated EuiDataGrid content to have a transparent background. (#8220)

Accessibility

  • When the tooltips components (EuiTooltip, EuiIconTip) are used inside components that handle the Escape key (like EuiFlyout or EuiModal), pressing the Escape key will now only close the tooltip and not the entire wrapping component. (#8140)
  • Improved the accessibility of EuiCodeBlocks by (#8195)
    • adding screen reader only labels
    • adding role="dialog" on in fullscreen mode
    • ensuring focus is returned on closing fullscreen mode

Borealis updates

  • [Visual Refresh] Update color token mappings (#8211)
  • [Visual Refresh] Introduce shared popover arrow styles to Borealis (#8212)
  • [Visual Refresh] Add forms.maxWidth token (#8221)
  • [Visual Refresh] Set darker shade for subdued text (#8224)
  • [Visual Refresh] Remove support for accentSecondary badges (#8224)
  • [Visual Refresh] Add severity vis colors (#8247)
  • [Visual Refresh] Fix transparent color variable definitions (8249)
  • [Visual Refresh] Update EuiToken colors (8250)

@mgadewoll mgadewoll added release_note:skip Skip the PR/issue when compiling release notes EUI backport:skip This commit does not require backporting v9.0.0 labels Dec 16, 2024
@mgadewoll mgadewoll changed the title Upgrade EUI to v98.2.0 Upgrade EUI to v98.2.1 Dec 17, 2024
@mgadewoll mgadewoll force-pushed the eui/v98.2.0 branch 2 times, most recently from b9d0657 to 5b9c720 Compare December 18, 2024 08:37
@eokoneyo eokoneyo mentioned this pull request Dec 18, 2024
Open
@mgadewoll mgadewoll marked this pull request as ready for review December 18, 2024 13:00
@mgadewoll mgadewoll requested review from a team as code owners December 18, 2024 13:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/eui-team (EUI)

afharo
afharo approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

Core changes LGTM (translation labels)

nreese
nreese approved these changes Dec 18, 2024
View reviewed changes
Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

kibana-presentation changes LGTM. Changes only effect test snapshots

consulthys
consulthys approved these changes Dec 18, 2024
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

eokoneyo
eokoneyo approved these changes Dec 19, 2024
View reviewed changes
Copy link
Contributor

@eokoneyo eokoneyo left a comment

Choose a reason for hiding this comment

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

Shared UX changes LGTM

@elastic-vault-github-plugin-prod elastic-vault-github-plugin-prod bot requested a review from a team as a code owner December 19, 2024 15:00
mattkime
mattkime approved these changes Dec 19, 2024
View reviewed changes
Copy link
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

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

code owner changes lgtm

@elasticmachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

Metrics [docs]

‼️ ERROR: no builds found for mergeBase sha [f881df1]

History

azasypkin
azasypkin approved these changes Dec 19, 2024
View reviewed changes
Copy link
Member

@azasypkin azasypkin left a comment

Choose a reason for hiding this comment

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

Changes in platform security related snapshots LGTM.

jughosta
jughosta approved these changes Dec 19, 2024
View reviewed changes
Copy link
Contributor

@jughosta jughosta left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mgadewoll mgadewoll requested a review from jughosta December 19, 2024 16:38
@mgadewoll mgadewoll mentioned this pull request Dec 19, 2024
Draft
@ryankeairns ryankeairns mentioned this pull request Dec 19, 2024
Closed
@JasonStoltz JasonStoltz removed the request for review from a team December 19, 2024 19:56
@mgadewoll mgadewoll merged commit a0ebb1d into elastic:main Dec 19, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benakansara benakansara benakansara approved these changes

@viduni94 viduni94 viduni94 approved these changes

@Ikuni17 Ikuni17 Ikuni17 approved these changes

@eokoneyo eokoneyo eokoneyo approved these changes

@afharo afharo afharo approved these changes

@azasypkin azasypkin azasypkin approved these changes

@jughosta jughosta jughosta approved these changes

@consulthys consulthys consulthys approved these changes

@nreese nreese nreese approved these changes

@mattkime mattkime mattkime approved these changes

Assignees
No one assigned
Labels
backport:skip This commit does not require backporting EUI release_note:skip Skip the PR/issue when compiling release notes v9.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.