Skip to content

[8.18] [ObsUX][A11y] Add aria-label to group by count (#217302)#217449

Merged
MiriamAparicio merged 1 commit into
elastic:8.18from
MiriamAparicio:backport/8.18/pr-217302
Apr 8, 2025
Merged

[8.18] [ObsUX][A11y] Add aria-label to group by count (#217302)#217449
MiriamAparicio merged 1 commit into
elastic:8.18from
MiriamAparicio:backport/8.18/pr-217302

Conversation

@MiriamAparicio
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

Closes elastic#194974

### What was done

- for the `All` button a description and spell out was done in
elastic#216592
- added aria-label to count number so it spells out the number of type
of node
<img width="460" alt="Screenshot 2025-04-07 at 11 26 32"
src="https://github.com/user-attachments/assets/7b0e814f-7713-4a6c-9b6c-2a45bb9b28c5"
/>
<img width="460" alt="Screenshot 2025-04-07 at 11 26 15"
src="https://github.com/user-attachments/assets/33ca56d4-09e6-4b5a-be36-7771b1467272"
/>
<img width="460" alt="Screenshot 2025-04-07 at 11 25 51"
src="https://github.com/user-attachments/assets/2c8bcf14-6308-4c22-a76a-b8313f2f580e"
/>
<img width="460" alt="Screenshot 2025-04-07 at 11 24 47"
src="https://github.com/user-attachments/assets/10497f74-77ba-48a9-ac2d-d3cbdcbeb609"
/>

(cherry picked from commit 340c86e)

# Conflicts:
#	x-pack/solutions/observability/plugins/infra/public/pages/metrics/inventory_view/components/waffle/group_name.tsx
@MiriamAparicio MiriamAparicio added the backport This PR is a backport of another PR label Apr 8, 2025
@MiriamAparicio MiriamAparicio enabled auto-merge (squash) April 8, 2025 07:57
@MiriamAparicio MiriamAparicio merged commit d2b827b into elastic:8.18 Apr 8, 2025
@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
infra 1.5MB 1.5MB +206.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