Skip to content

[9.3] [Search] Replace aria-selected with aria-expanded on Result toggle button (#260561)#260581

Merged
kibanamachine merged 1 commit into
elastic:9.3from
kibanamachine:backport/9.3/pr-260561
Mar 31, 2026
Merged

[9.3] [Search] Replace aria-selected with aria-expanded on Result toggle button (#260561)#260581
kibanamachine merged 1 commit into
elastic:9.3from
kibanamachine:backport/9.3/pr-260561

Conversation

@kibanamachine
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

…tton (elastic#260561)

## Summary

## Summary

Fixes a critical `aria-allowed-attr` accessibility violation in the
`Result` component (`@kbn/search-index-documents`).

The "Show more fields" `EuiButtonIcon` was using `aria-selected` to
communicate its toggle state, but `aria-selected` is not a valid ARIA
attribute for the `button` role. This was replaced with `aria-expanded`,
which is the semantically correct attribute for a button that controls
the visibility of related content.

(cherry picked from commit 70d4f49)
@kibanamachine kibanamachine merged commit b754300 into elastic:9.3 Mar 31, 2026
19 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] src/platform/test/functional/apps/discover/esql/config.ts / discover/esql discover esql controls when adding an ES|QL panel with controls in dashboards and exploring it in discover should retain the controls and their state

Metrics [docs]

✅ unchanged

cc @seialkali

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