Skip to content

[9.2] Address @elastic/eui/require-table-caption lint violations across @elastic/actionable-obs-team files (#250682)#253027

Merged
alexwizp merged 8 commits intoelastic:9.2from
alexwizp:backport/9.2/pr-250682
Feb 18, 2026
Merged

[9.2] Address @elastic/eui/require-table-caption lint violations across @elastic/actionable-obs-team files (#250682)#253027
alexwizp merged 8 commits intoelastic:9.2from
alexwizp:backport/9.2/pr-250682

Conversation

@alexwizp
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

…lastic/actionable-obs-team` files (elastic#250682)

> [!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: Miguel Martín <miguel.martin@elastic.co>
(cherry picked from commit 29698cf)

# Conflicts:
#	x-pack/solutions/observability/plugins/uptime/public/legacy_uptime/components/certificates/certificates_list.tsx
@alexwizp alexwizp added the backport This PR is a backport of another PR label Feb 13, 2026
@alexwizp alexwizp enabled auto-merge (squash) February 13, 2026 10:25
@alexwizp alexwizp added the backport This PR is a backport of another PR label Feb 13, 2026
@botelastic botelastic Bot added Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation Team:obs-ux-management labels Feb 13, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-presentation-team (Team:obs-presentation)

@bhavyarm bhavyarm self-requested a review February 13, 2026 11:14
Copy link
Copy Markdown
Contributor

@bhavyarm bhavyarm left a comment

Choose a reason for hiding this comment

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

LGTM

@alexwizp alexwizp merged commit 6b5a5b9 into elastic:9.2 Feb 18, 2026
13 checks passed
@elasticmachine
Copy link
Copy Markdown
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
observability 1.6MB 1.6MB +120.0B
slo 982.2KB 982.4KB +234.0B
synthetics 1.0MB 1.0MB +976.0B
uptime 492.3KB 492.8KB +570.0B
ux 137.1KB 137.1KB +93.0B
total +1.9KB

History

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:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation Team:obs-ux-management

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants