Skip to content

[9.3] Add aria-hidden to decorative EuiIcon components in ES|QL files (#262744)#262781

Merged
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-262744
Apr 13, 2026
Merged

[9.3] Add aria-hidden to decorative EuiIcon components in ES|QL files (#262744)#262781
kibanamachine merged 1 commit intoelastic:9.3from
kibanamachine:backport/9.3/pr-262744

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

…lastic#262744)

Closes: elastic#262712

Fixes 3 `@elastic/eui/icon-accessibility-rules` lint violations across
`@elastic/kibana-esql` files. All three icons are decorative (adjacent
text already conveys meaning), so `aria-hidden={true}` is the correct
fix.

- **`errors_warnings_popover.tsx`** — error/warning icon next to "Line
N" text
- **`column_header_renderer.tsx`** — plus icon next to "Add a field…"
label
- **`data_grid.tsx`** — Discover app icon inside "Open in Discover" link

Closes elastic#224193

> [!WARNING]
>
> <details>
> <summary>Firewall rules blocked me from connecting to one or more
addresses (expand for details)</summary>
>
> #### I tried to connect to the following addresses, but was blocked by
firewall rules:
>
> - `ci-stats.kibana.dev`
> - Triggering command:
`/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node
/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node
scripts/yarn_install_scripts.js run ldd 0.8.2` (dns block)
> - Triggering command:
`/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node
/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node scripts/kbn
bootstrap` (dns block)
> - Triggering command:
`/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node
/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node scripts/kbn
bootstrap --no-validate s/li��
t/webkit-2272/minibrowser-gtk/sys/lib/libbrotlidec.so.1 t }
  }
{ printf &elastic#34;%s%s&elastic#34;, sep, --compressed /ldd get --global t ldd
s/li��` (dns block)
> - `clients3.google.com`
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd b/li��` (dns block)
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd s/li��
t/webkit-2272/minibrowser-gtk/sys/lib/libbrotlidec.so.1.0.7 git
v22.22.0/lib/node_modules/npm/bin/node-gyp-bin/ldd get --global
cal/bin/git ldd nibr��` (dns block)
> - `detectportal.firefox.com`
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd b/li��` (dns block)
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd s/li��
t/webkit-2272/minibrowser-gtk/sys/lib/libbrotlidec.so.1.0.7 git
v22.22.0/lib/node_modules/npm/bin/node-gyp-bin/ldd get --global
cal/bin/git ldd nibr��` (dns block)
> - `google.com`
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd b/li��` (dns block)
> - Triggering command:
`/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
/home/REDACTED/work/kibana/kibana/node_modules/@moonrepo/core-linux-x64-gnu/moon
run :build-webpack ldd s/li��
t/webkit-2272/minibrowser-gtk/sys/lib/libbrotlidec.so.1.0.7 git
v22.22.0/lib/node_modules/npm/bin/node-gyp-bin/ldd get --global
cal/bin/git ldd nibr��` (dns block)
> - `googlechromelabs.github.io`
> - Triggering command:
`/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node
/home/REDACTED/.nvm/versions/node/v22.22.0/bin/node install.js
k/_temp/ghcca-node/node/bin/git ldd b/li��` (dns block)
> - `iojs.org`
> - Triggering command: `/usr/bin/curl curl -q --fail --compressed -L -s
REDACTED -o -` (dns block)
>
> If you need me to access, download, or install something from one of
these locations, you can either:
>
> - Configure [Actions setup
steps](https://gh.io/copilot/actions-setup-steps) to set up my
environment, which run before the firewall is enabled
> - Add the appropriate URLs or hosts to the custom allowlist in this
repository's [Copilot coding agent
settings](https://github.com/elastic/kibana/settings/copilot/coding_agent)
(admins only)
>
> </details>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexwizp <20072247+alexwizp@users.noreply.github.com>
(cherry picked from commit cc0eea1)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 13, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) April 13, 2026 13:57
@kibanamachine kibanamachine merged commit 1084e46 into elastic:9.3 Apr 13, 2026
15 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
esql 716.3KB 716.3KB +34.0B
esqlDataGrid 151.5KB 151.5KB +17.0B
total +51.0B

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