Skip to content

Comments

Address @elastic/eui/require-table-caption lint violations across @elastic/contextual-security-apps files#250018

Merged
alexwizp merged 6 commits intoelastic:mainfrom
alexwizp:jan-22-2
Jan 29, 2026
Merged

Address @elastic/eui/require-table-caption lint violations across @elastic/contextual-security-apps files#250018
alexwizp merged 6 commits intoelastic:mainfrom
alexwizp:jan-22-2

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 Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.3.0 v9.4.0 v9.2.5 labels Jan 22, 2026
@alexwizp alexwizp marked this pull request as ready for review January 23, 2026 15:58
@alexwizp alexwizp requested a review from a team as a code owner January 23, 2026 15:58
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout: [ platform / fleet ] plugin / Integrations are visible but cannot be added
  • [job] [logs] Scout: [ platform / fleet ] plugin / stateful - When the user has All privileges for Integrations but None for Fleet - Integrations are visible but cannot be added

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
cloudSecurityPosture 657.8KB 658.6KB +899.0B
securitySolution 10.8MB 10.8MB +553.0B
total +1.4KB

History

@alexwizp alexwizp merged commit 9e5c23c into elastic:main Jan 29, 2026
16 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2, 9.3

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

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Jan 29, 2026
…@elastic/contextual-security-apps` files (elastic#250018)

> [!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 9e5c23c)
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Jan 29, 2026
…@elastic/contextual-security-apps` files (elastic#250018)

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

💚 All backports created successfully

Status Branch Result
9.2
9.3

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 29, 2026
…ross `@elastic/contextual-security-apps` files (#250018) (#250918)

# Backport

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

<!--- 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-29T14:49:35Z","message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","backport:version","v9.3.0","v9.4.0","v9.2.5"],"title":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps`
files","number":250018,"url":"https://github.com/elastic/kibana/pull/250018","mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695"}},"sourceBranch":"main","suggestedTargetBranches":["9.3","9.2"],"targetPullRequestStates":[{"branch":"9.3","label":"v9.3.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250018","number":250018,"mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695"}},{"branch":"9.2","label":"v9.2.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
kibanamachine added a commit that referenced this pull request Jan 29, 2026
…ross `@elastic/contextual-security-apps` files (#250018) (#250919)

# Backport

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

<!--- 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-29T14:49:35Z","message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","backport:version","v9.3.0","v9.4.0","v9.2.5"],"title":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps`
files","number":250018,"url":"https://github.com/elastic/kibana/pull/250018","mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695"}},"sourceBranch":"main","suggestedTargetBranches":["9.3","9.2"],"targetPullRequestStates":[{"branch":"9.3","label":"v9.3.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250018","number":250018,"mergeCommit":{"message":"Address
`@elastic/eui/require-table-caption` lint violations across
`@elastic/contextual-security-apps` files (#250018)\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":"9e5c23ce54ce1b48a5f49235795a4f122cf75695"}},{"branch":"9.2","label":"v9.2.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
hannahbrooks pushed a commit to hannahbrooks/kibana that referenced this pull request Jan 30, 2026
…@elastic/contextual-security-apps` files (elastic#250018)

> [!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>
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.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants