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.

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

Upgrade EUI to v97.0.0 #195525

Merged
merged 13 commits into from
Oct 14, 2024
Merged

Upgrade EUI to v97.0.0 #195525

merged 13 commits into from
Oct 14, 2024

Conversation

cee-chen
Copy link
Member

@cee-chen cee-chen commented Oct 9, 2024

v96.1.0v97.0.0

Questions? Please see our Kibana upgrade FAQ.


v97.0.0

Breaking changes

  • EuiDataGrid's custom grid body (rendered via renderCustomGridBody) no longer automatically renders the column header row or footer rows. It instead now passes the headerRow and footerRow React elements, which require manual rendering. (#8028)
    • This change was made to allow consumers to sync header/footer rows with their own custom virtualization libraries.
    • To facilitate this, a gridWidth prop is now also passed to custom grid body renderers.

Bug fixes

  • Fixed inputs not taking the whole width when passing fullWidth as true to EuiDatePickerRange component (#8061)

Accessibility

  • Improved accessibility of EuiExternalLinkIcon by clarifying text for Screen Reader users. (#8065)

@cee-chen cee-chen marked this pull request as ready for review October 9, 2024 22:48
@cee-chen cee-chen requested review from a team as code owners October 9, 2024 22:48
@elasticmachine
Copy link
Contributor

Pinging @elastic/eui-team (EUI)

@cee-chen cee-chen requested review from a team as code owners October 9, 2024 22:48
Copy link
Member

@markov00 markov00 left a comment

Choose a reason for hiding this comment

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

checked the class name change in vislib legend and looks fine to me

@tkajtoch
Copy link
Member

We're experiencing some CI flakiness. I confirmed that none of these test failures are caused by the changes in this PR

Copy link
Member

@umbopepato umbopepato left a comment

Choose a reason for hiding this comment

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

The alerts table with custom grid body works correctly, LGTM!

Copy link
Member

@nickpeihl nickpeihl 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!

code review

@kc13greiner kc13greiner self-requested a review October 11, 2024 12:58
Copy link
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

LGTM!

@tkajtoch
Copy link
Member

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 14, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: c37fc99
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-195525-c37fc99aea48

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
dashboard 637.3KB 637.3KB -2.0B
maps 3.0MB 3.0MB -2.0B
securitySolution 20.8MB 20.8MB +100.0B
triggersActionsUi 1.6MB 1.6MB +61.0B
total +157.0B

Page load bundle

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

id before after diff
core 449.8KB 449.9KB +59.0B
kbnUiSharedDeps-npmDll 5.8MB 5.8MB +608.0B
total +667.0B

History

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 code changes LGTM 🚀

Copy link
Contributor

@szwarckonrad szwarckonrad left a comment

Choose a reason for hiding this comment

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

DW changes LGTM, thanks!

@delanni delanni merged commit cd60c66 into elastic:main Oct 14, 2024
54 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11326723307

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 14, 2024
`v96.1.0`⏩`v97.0.0`

_[Questions? Please see our Kibana upgrade
FAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_

---

## [`v97.0.0`](https://github.com/elastic/eui/releases/v97.0.0)

**Breaking changes**

- EuiDataGrid's custom grid body (rendered via `renderCustomGridBody`)
no longer automatically renders the column header row or footer rows. It
instead now passes the `headerRow` and `footerRow` React elements, which
require manual rendering.
([elastic#8028](elastic/eui#8028))
- This change was made to allow consumers to sync header/footer rows
with their own custom virtualization libraries.
- To facilitate this, a `gridWidth` prop is now also passed to custom
grid body renderers.

**Bug fixes**

- Fixed inputs not taking the whole width when passing `fullWidth` as
`true` to EuiDatePickerRange component
([elastic#8061](elastic/eui#8061))

**Accessibility**

- Improved accessibility of `EuiExternalLinkIcon` by clarifying text for
Screen Reader users. ([elastic#8065](elastic/eui#8065))

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit cd60c66)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 14, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Upgrade EUI to v97.0.0
(#195525)](#195525)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Cee
Chen","email":"549407+cee-chen@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-10-14T11:35:35Z","message":"Upgrade
EUI to v97.0.0 (#195525)\n\n`v96.1.0`⏩`v97.0.0`\r\n\r\n_[Questions?
Please see our Kibana
upgrade\r\nFAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_\r\n\r\n---\r\n\r\n##
[`v97.0.0`](https://github.com/elastic/eui/releases/v97.0.0)\r\n\r\n**Breaking
changes**\r\n\r\n- EuiDataGrid's custom grid body (rendered via
`renderCustomGridBody`)\r\nno longer automatically renders the column
header row or footer rows. It\r\ninstead now passes the `headerRow` and
`footerRow` React elements, which\r\nrequire manual
rendering.\r\n([#8028](https://github.com/elastic/eui/pull/8028))\r\n-
This change was made to allow consumers to sync header/footer
rows\r\nwith their own custom virtualization libraries.\r\n- To
facilitate this, a `gridWidth` prop is now also passed to custom\r\ngrid
body renderers.\r\n\r\n**Bug fixes**\r\n\r\n- Fixed inputs not taking
the whole width when passing `fullWidth` as\r\n`true` to
EuiDatePickerRange
component\r\n([#8061](https://github.com/elastic/eui/pull/8061))\r\n\r\n**Accessibility**\r\n\r\n-
Improved accessibility of `EuiExternalLinkIcon` by clarifying text
for\r\nScreen Reader users.
([#8065](https://github.com/elastic/eui/pull/8065))\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"cd60c66d19ffdb4130a5c2d1b76e80cfaf8cde78","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Embedding","release_note:skip","EUI","v9.0.0","ci:project-deploy-observability","Team:obs-ux-infra_services","Team:obs-ux-management","v8.16.0","backport:version"],"title":"Upgrade
EUI to
v97.0.0","number":195525,"url":"https://github.com/elastic/kibana/pull/195525","mergeCommit":{"message":"Upgrade
EUI to v97.0.0 (#195525)\n\n`v96.1.0`⏩`v97.0.0`\r\n\r\n_[Questions?
Please see our Kibana
upgrade\r\nFAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_\r\n\r\n---\r\n\r\n##
[`v97.0.0`](https://github.com/elastic/eui/releases/v97.0.0)\r\n\r\n**Breaking
changes**\r\n\r\n- EuiDataGrid's custom grid body (rendered via
`renderCustomGridBody`)\r\nno longer automatically renders the column
header row or footer rows. It\r\ninstead now passes the `headerRow` and
`footerRow` React elements, which\r\nrequire manual
rendering.\r\n([#8028](https://github.com/elastic/eui/pull/8028))\r\n-
This change was made to allow consumers to sync header/footer
rows\r\nwith their own custom virtualization libraries.\r\n- To
facilitate this, a `gridWidth` prop is now also passed to custom\r\ngrid
body renderers.\r\n\r\n**Bug fixes**\r\n\r\n- Fixed inputs not taking
the whole width when passing `fullWidth` as\r\n`true` to
EuiDatePickerRange
component\r\n([#8061](https://github.com/elastic/eui/pull/8061))\r\n\r\n**Accessibility**\r\n\r\n-
Improved accessibility of `EuiExternalLinkIcon` by clarifying text
for\r\nScreen Reader users.
([#8065](https://github.com/elastic/eui/pull/8065))\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"cd60c66d19ffdb4130a5c2d1b76e80cfaf8cde78"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195525","number":195525,"mergeCommit":{"message":"Upgrade
EUI to v97.0.0 (#195525)\n\n`v96.1.0`⏩`v97.0.0`\r\n\r\n_[Questions?
Please see our Kibana
upgrade\r\nFAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_\r\n\r\n---\r\n\r\n##
[`v97.0.0`](https://github.com/elastic/eui/releases/v97.0.0)\r\n\r\n**Breaking
changes**\r\n\r\n- EuiDataGrid's custom grid body (rendered via
`renderCustomGridBody`)\r\nno longer automatically renders the column
header row or footer rows. It\r\ninstead now passes the `headerRow` and
`footerRow` React elements, which\r\nrequire manual
rendering.\r\n([#8028](https://github.com/elastic/eui/pull/8028))\r\n-
This change was made to allow consumers to sync header/footer
rows\r\nwith their own custom virtualization libraries.\r\n- To
facilitate this, a `gridWidth` prop is now also passed to custom\r\ngrid
body renderers.\r\n\r\n**Bug fixes**\r\n\r\n- Fixed inputs not taking
the whole width when passing `fullWidth` as\r\n`true` to
EuiDatePickerRange
component\r\n([#8061](https://github.com/elastic/eui/pull/8061))\r\n\r\n**Accessibility**\r\n\r\n-
Improved accessibility of `EuiExternalLinkIcon` by clarifying text
for\r\nScreen Reader users.
([#8065](https://github.com/elastic/eui/pull/8065))\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"cd60c66d19ffdb4130a5c2d1b76e80cfaf8cde78"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Cee Chen <549407+cee-chen@users.noreply.github.com>
@cee-chen cee-chen deleted the eui/v97.0.0 branch October 14, 2024 20:40
maximpn added a commit that referenced this pull request Oct 16, 2024
…6309)

## Summary

This PR generalizes external link text cleanup in `removeExternalLinkText`. 

## Details

Recent Rule Management [periodic pipeline failure](https://buildkite.com/elastic/security-serverless-quality-gate-kibana-periodic/builds/1209#job-01928e56-28f3-4b45-8f9f-7158b324c115) was caused by merging back #195525. Despite EUI changes were [addressed](https://github.com/elastic/kibana/pull/195525/files#diff-8d47b006a91beb2c5074560dbcd42eecef96173e03ffeec7c726dd322425f760) in our test utility it wasn't properly picked up. The problem is fixed in a more general way.
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 16, 2024
…stic#196309)

## Summary

This PR generalizes external link text cleanup in `removeExternalLinkText`.

## Details

Recent Rule Management [periodic pipeline failure](https://buildkite.com/elastic/security-serverless-quality-gate-kibana-periodic/builds/1209#job-01928e56-28f3-4b45-8f9f-7158b324c115) was caused by merging back elastic#195525. Despite EUI changes were [addressed](https://github.com/elastic/kibana/pull/195525/files#diff-8d47b006a91beb2c5074560dbcd42eecef96173e03ffeec7c726dd322425f760) in our test utility it wasn't properly picked up. The problem is fixed in a more general way.

(cherry picked from commit daf4aae)
kibanamachine added a commit that referenced this pull request Oct 16, 2024
#196309) (#196502)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution] Clean up external link text in a general way
(#196309)](#196309)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Maxim
Palenov","email":"maxim.palenov@elastic.co"},"sourceCommit":{"committedDate":"2024-10-16T09:00:18Z","message":"[Security
Solution] Clean up external link text in a general way (#196309)\n\n##
Summary\r\n\r\nThis PR generalizes external link text cleanup in
`removeExternalLinkText`. \r\n\r\n## Details\r\n\r\nRecent Rule
Management [periodic pipeline
failure](https://buildkite.com/elastic/security-serverless-quality-gate-kibana-periodic/builds/1209#job-01928e56-28f3-4b45-8f9f-7158b324c115)
was caused by merging back
#195525. Despite EUI changes were
[addressed](https://github.com/elastic/kibana/pull/195525/files#diff-8d47b006a91beb2c5074560dbcd42eecef96173e03ffeec7c726dd322425f760)
in our test utility it wasn't properly picked up. The problem is fixed
in a more general
way.","sha":"daf4aae6aa202fd1148beea269939090137b659a","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","v9.0.0","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","backport:prev-minor","v8.17.0"],"title":"[Security
Solution] Clean up external link text in a general
way","number":196309,"url":"https://github.com/elastic/kibana/pull/196309","mergeCommit":{"message":"[Security
Solution] Clean up external link text in a general way (#196309)\n\n##
Summary\r\n\r\nThis PR generalizes external link text cleanup in
`removeExternalLinkText`. \r\n\r\n## Details\r\n\r\nRecent Rule
Management [periodic pipeline
failure](https://buildkite.com/elastic/security-serverless-quality-gate-kibana-periodic/builds/1209#job-01928e56-28f3-4b45-8f9f-7158b324c115)
was caused by merging back
#195525. Despite EUI changes were
[addressed](https://github.com/elastic/kibana/pull/195525/files#diff-8d47b006a91beb2c5074560dbcd42eecef96173e03ffeec7c726dd322425f760)
in our test utility it wasn't properly picked up. The problem is fixed
in a more general
way.","sha":"daf4aae6aa202fd1148beea269939090137b659a"}},"sourceBranch":"main","suggestedTargetBranches":["8.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196309","number":196309,"mergeCommit":{"message":"[Security
Solution] Clean up external link text in a general way (#196309)\n\n##
Summary\r\n\r\nThis PR generalizes external link text cleanup in
`removeExternalLinkText`. \r\n\r\n## Details\r\n\r\nRecent Rule
Management [periodic pipeline
failure](https://buildkite.com/elastic/security-serverless-quality-gate-kibana-periodic/builds/1209#job-01928e56-28f3-4b45-8f9f-7158b324c115)
was caused by merging back
#195525. Despite EUI changes were
[addressed](https://github.com/elastic/kibana/pull/195525/files#diff-8d47b006a91beb2c5074560dbcd42eecef96173e03ffeec7c726dd322425f760)
in our test utility it wasn't properly picked up. The problem is fixed
in a more general
way.","sha":"daf4aae6aa202fd1148beea269939090137b659a"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Maxim Palenov <maxim.palenov@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:review backport:version Backport to applied version labels ci:project-deploy-observability Create an Observability project EUI Feature:Embedding Embedding content via iFrame release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team Team:obs-ux-management Observability Management User Experience Team v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.