Skip to content

[9.3] Address @elastic/eui/require-table-caption lint violations across @elastic/fleet files (#250004)#250082

Merged
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-250004
Jan 22, 2026
Merged

[9.3] Address @elastic/eui/require-table-caption lint violations across @elastic/fleet files (#250004)#250082
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-250004

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

…@elastic/fleet` files (elastic#250004)

> [!CAUTION]
> ⚠️ **Changes / translations were made by GenAI**. I’ve reviewed them
carefully, but your code owners’ expert eyes will ensure they’re 100%
right.

## Summary
This PR applies the auto-fix for the newly introduced
`@elastic/eui/require-table-caption`.
This rule ensure `EuiInMemoryTable`, `EuiBasicTable` have a
`tableCaption` prop for accessibility.

## Changes

1. 🎯 Added missing `tableCaption` attributes to elements flagged by
`@elastic/eui/require-table-caption` — accessibility leveled up!

## Related
- elastic/eui#9168

This time, to avoid annoying approvals collection, we've broken files
down by teams. Now, we are waiting a review only from your team!

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit c4e692d)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jan 22, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) January 22, 2026 16:17
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Jan 22, 2026
@kibanamachine kibanamachine merged commit 8f889dd into elastic:9.3 Jan 22, 2026
19 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #25 / discover - group 1 Discover CSV Export Generate CSV: sparse data handles field formatting for a field that doesn't exist initially

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
fleet 2.1MB 2.1MB +1.9KB

Page load bundle

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

id before after diff
fleet 180.8KB 181.1KB +291.0B

cc @alexwizp

alexwizp added a commit that referenced this pull request Jan 26, 2026
…ross `@elastic/fleet` files (#250004) (#250166)

# Backport

This will backport the following commits from `main` to `9.2`:
- [Address `@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet` files
(#250004)](#250004)

<!--- Backport version: 10.2.0 -->

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

<!--BACKPORT [{"author":{"name":"Alexey
Antonov","email":"alexwizp@gmail.com"},"sourceCommit":{"committedDate":"2026-01-22T16:01:41Z","message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet` files (#250004)\n\n> [!CAUTION]\n> ⚠️ **Changes /
translations were made by GenAI**. I’ve reviewed them\ncarefully, but
your code owners’ expert eyes will ensure they’re 100%\nright.\n\n##
Summary\nThis PR applies the auto-fix for the newly
introduced\n`@elastic/eui/require-table-caption`.\nThis rule ensure
`EuiInMemoryTable`, `EuiBasicTable` have a\n`tableCaption` prop for
accessibility.\n\n## Changes\n\n1. 🎯 Added missing `tableCaption`
attributes to elements flagged by\n`@elastic/eui/require-table-caption`
— accessibility leveled up!\n\n## Related\n-
https://github.com/elastic/eui/pull/9168\n\nThis time, to avoid annoying
approvals collection, we've broken files\ndown by teams. Now, we are
waiting a review only from your team!\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"c4e692de0688c5588961ef06045f8812b15128bc","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","Team:Fleet","backport:version","v9.3.0","v9.4.0","v9.2.5","v9.3.1"],"title":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet`
files","number":250004,"url":"https://github.com/elastic/kibana/pull/250004","mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet` files (#250004)\n\n> [!CAUTION]\n> ⚠️ **Changes /
translations were made by GenAI**. I’ve reviewed them\ncarefully, but
your code owners’ expert eyes will ensure they’re 100%\nright.\n\n##
Summary\nThis PR applies the auto-fix for the newly
introduced\n`@elastic/eui/require-table-caption`.\nThis rule ensure
`EuiInMemoryTable`, `EuiBasicTable` have a\n`tableCaption` prop for
accessibility.\n\n## Changes\n\n1. 🎯 Added missing `tableCaption`
attributes to elements flagged by\n`@elastic/eui/require-table-caption`
— accessibility leveled up!\n\n## Related\n-
https://github.com/elastic/eui/pull/9168\n\nThis time, to avoid annoying
approvals collection, we've broken files\ndown by teams. Now, we are
waiting a review only from your team!\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"c4e692de0688c5588961ef06045f8812b15128bc"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"9.3","label":"v9.3.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/250082","number":250082,"state":"MERGED","mergeCommit":{"sha":"8f889dd92ea44dbc166ac76d5279aa0f6b1fd807","message":"[9.3]
Address `@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet` files (#250004) (#250082)\n\n# Backport\n\nThis will
backport the following commits from `main` to `9.3`:\n- [Address
`@elastic/eui/require-table-caption` lint violations
across\n`@elastic/fleet`
files\n(#250004)](https://github.com/elastic/kibana/pull/250004)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Alexey Antonov
<alexwizp@gmail.com>"}},{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250004","number":250004,"mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/fleet` files (#250004)\n\n> [!CAUTION]\n> ⚠️ **Changes /
translations were made by GenAI**. I’ve reviewed them\ncarefully, but
your code owners’ expert eyes will ensure they’re 100%\nright.\n\n##
Summary\nThis PR applies the auto-fix for the newly
introduced\n`@elastic/eui/require-table-caption`.\nThis rule ensure
`EuiInMemoryTable`, `EuiBasicTable` have a\n`tableCaption` prop for
accessibility.\n\n## Changes\n\n1. 🎯 Added missing `tableCaption`
attributes to elements flagged by\n`@elastic/eui/require-table-caption`
— accessibility leveled up!\n\n## Related\n-
https://github.com/elastic/eui/pull/9168\n\nThis time, to avoid annoying
approvals collection, we've broken files\ndown by teams. Now, we are
waiting a review only from your team!\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"c4e692de0688c5588961ef06045f8812b15128bc"}},{"branch":"9.2","label":"v9.2.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:Fleet Team label for Observability Data Collection Fleet team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants