Skip to content

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

Merged
alexwizp merged 3 commits intoelastic:9.3from
alexwizp:backport/9.3/pr-250682
Feb 16, 2026
Merged

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

Conversation

@alexwizp
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.3:

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:24
@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-presentation-team (Team:obs-presentation)

@elasticmachine
Copy link
Copy Markdown
Contributor

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

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

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 13, 2026

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #5 / AvailabilityReporting component renders correctly against snapshot
  • [job] [logs] Jest Tests #5 / AvailabilityReporting component renders correctly against snapshot
  • [job] [logs] Jest Tests #5 / CertificatesPage renders expected elements for valid props
  • [job] [logs] Jest Tests #8 / CertificatesPage renders expected elements for valid props
  • [job] [logs] Jest Tests #5 / CertificatesPage renders expected elements for valid props
  • [job] [logs] Jest Tests #8 / CertificatesPage renders expected elements for valid props
  • [job] [logs] Jest Tests #8 / search_bar should render search bar and fetch index pattern
  • [job] [logs] Jest Tests #8 / search_bar should render search bar and fetch index pattern

History

@alexwizp alexwizp merged commit 9ee6d92 into elastic:9.3 Feb 16, 2026
13 checks passed
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