Skip to content

Address @elastic/eui/require-table-caption lint violations across @elastic/kibana-management files#250015

Merged
alexwizp merged 7 commits intoelastic:mainfrom
alexwizp:jan-22-1
Jan 27, 2026
Merged

Address @elastic/eui/require-table-caption lint violations across @elastic/kibana-management files#250015
alexwizp merged 7 commits intoelastic:mainfrom
alexwizp:jan-22-1

Conversation

@alexwizp
Copy link
Contributor

@alexwizp alexwizp commented Jan 22, 2026

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

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

@alexwizp alexwizp added v9.4.0 v9.2.5 v9.3.1 Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels labels Jan 22, 2026
@alexwizp alexwizp changed the title Address @elastic/eui/require-table-caption lint violations across @elastic/kibana-management files Address @elastic/eui/require-table-caption lint violations across @elastic/kibana-management files Jan 22, 2026
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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
console 199.2KB 199.4KB +115.0B
crossClusterReplication 139.3KB 139.6KB +290.0B
indexLifecycleManagement 146.2KB 146.5KB +321.0B
indexManagement 736.9KB 737.6KB +750.0B
ingestPipelines 349.2KB 349.5KB +262.0B
remoteClusters 84.3KB 84.4KB +125.0B
rollup 108.8KB 109.0KB +234.0B
searchprofiler 31.5KB 31.6KB +136.0B
snapshotRestore 251.9KB 252.5KB +615.0B
transform 624.8KB 625.2KB +404.0B
upgradeAssistant 226.9KB 227.2KB +349.0B
watcher 157.9KB 158.8KB +938.0B
total +4.4KB

History

@alexwizp alexwizp marked this pull request as ready for review January 26, 2026 13:17
@alexwizp alexwizp requested a review from a team as a code owner January 26, 2026 13:17
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-accessibility (Project:Accessibility)

Copy link
Contributor

@ElenaStoeva ElenaStoeva left a comment

Choose a reason for hiding this comment

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

Thanks for the change! Lgtm

@alexwizp alexwizp merged commit b3fa266 into elastic:main Jan 27, 2026
20 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2, 9.3

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

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Jan 27, 2026
…@elastic/kibana-management` files (elastic#250015)

> [!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 b3fa266)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
9.2 Backport failed because of merge conflicts
9.3

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 250015

Questions ?

Please refer to the Backport tool documentation

alexwizp added a commit to alexwizp/kibana that referenced this pull request Jan 27, 2026
…@elastic/kibana-management` files (elastic#250015)

> [!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 b3fa266)

# Conflicts:
#	x-pack/platform/plugins/private/watcher/public/application/sections/watch_status_page/components/action_statuses_panel.tsx
#	x-pack/platform/plugins/shared/index_management/public/application/sections/home/data_stream_list/data_stream_table/data_stream_table.tsx
#	x-pack/platform/plugins/shared/index_management/public/application/sections/home/template_list/legacy_templates/template_table/template_table.tsx
#	x-pack/platform/plugins/shared/index_management/public/application/sections/home/template_list/template_table/template_table.tsx
@alexwizp
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
9.2

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 Jan 27, 2026
…ross `@elastic/kibana-management` files (#250015) (#250548)

# Backport

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

<!--- Backport version: 9.6.6 -->

### 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-27T10:36:37Z","message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","backport:version","v9.4.0","v9.2.5","v9.3.1"],"title":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management`
files","number":250015,"url":"https://github.com/elastic/kibana/pull/250015","mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12"}},"sourceBranch":"main","suggestedTargetBranches":["9.2","9.3"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250015","number":250015,"mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12"}},{"branch":"9.2","label":"v9.2.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.3","label":"v9.3.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
alexwizp added a commit that referenced this pull request Jan 27, 2026
…ross `@elastic/kibana-management` files (#250015) (#250555)

# Backport

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

<!--- 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-27T10:36:37Z","message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","backport:version","v9.4.0","v9.2.5","v9.3.1"],"title":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management`
files","number":250015,"url":"https://github.com/elastic/kibana/pull/250015","mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250015","number":250015,"mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/kibana-management` files (#250015)\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":"b3fa266cdf123821bef40c1af99efeb603e49b12"}},{"branch":"9.2","label":"v9.2.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.3","label":"v9.3.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/250548","number":250548,"state":"OPEN"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes v9.2.5 v9.3.0 v9.3.1 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants