Skip to content

[9.4] [ML] Fix missing accessible labels on scatterplot matrix form controls (#264962)#266474

Merged
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/pr-264962
Apr 29, 2026
Merged

[9.4] [ML] Fix missing accessible labels on scatterplot matrix form controls (#264962)#266474
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/pr-264962

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.4:

Questions ?

Please refer to the Backport tool documentation

elastic#264962)

Closes: elastic#264955

The "Random scoring" and "Dynamic size" switches in the Results Explorer
scatterplot matrix announce as "Off, off, switch" to screen readers
instead of including their descriptive titles, violating WCAG 4.1.2
(Name, Role, Value).

- Set `label` to the descriptive title (`'Random scoring'` / `'Dynamic
size'`) with `showLabel={false}` so the label is visually hidden but
exposed to assistive technology. The switch role communicates on/off
state via `aria-checked`.
- Removed the `OptionLabelWithIconTip` component and moved tooltip icons
into `EuiFormRow`'s `labelAppend` prop, so all form row labels are plain
text strings with proper accessibility association.
- Removed now-unused `TOGGLE_ON`/`TOGGLE_OFF` constants and their
translation entries.

Screen readers will now announce: **"Random scoring, off, switch"**.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexwizp <20072247+alexwizp@users.noreply.github.com>
Co-authored-by: Alexey Antonov <alexwizp@gmail.com>
(cherry picked from commit 622954a)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 29, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) April 29, 2026 17:09
@kibanamachine
Copy link
Copy Markdown
Contributor Author

💚 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
ml 5.8MB 5.8MB -144.0B

@kibanamachine kibanamachine merged commit 2b3fb92 into elastic:9.4 Apr 29, 2026
27 of 28 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants