Skip to content

[9.3] Address @elastic/eui/require-table-caption lint violations across @elastic/ml-ui files (#249699)#250249

Merged
kibanamachine merged 4 commits intoelastic:9.3from
kibanamachine:backport/9.3/pr-249699
Jan 30, 2026
Merged

[9.3] Address @elastic/eui/require-table-caption lint violations across @elastic/ml-ui files (#249699)#250249
kibanamachine merged 4 commits intoelastic:9.3from
kibanamachine:backport/9.3/pr-249699

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/ml-ui` files (elastic#249699)

> [!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>
Co-authored-by: Pete Harverson <pete@elastic.co>
(cherry picked from commit 2e91ece)
@kibanamachine kibanamachine merged commit 0e589c0 into elastic:9.3 Jan 30, 2026
13 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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
aiops 527.5KB 528.0KB +544.0B
dataVisualizer 601.3KB 601.4KB +119.0B
esql 715.4KB 715.5KB +119.0B
ml 5.6MB 5.6MB +3.9KB
total +4.7KB

History

cc @alexwizp

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants