Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[AXE-CORE][Serverless - Observability] Observability -> SLOs -> Metric sparkline charts fail <svg> elements with an img role must have an alternative text a11y check #179491

Closed
bhavyarm opened this issue Mar 26, 2024 · 2 comments · Fixed by #183040
Closed

[AXE-CORE][Serverless - Observability] Observability -> SLOs -> Metric sparkline charts fail <svg> elements with an img role must have an alternative text a11y check #179491

bhavyarm opened this issue Mar 26, 2024 · 2 comments · Fixed by #183040
Assignees
@alexwizp
Labels
bug Fixes for quality problems that affect the customer experience defect-level-1 Critical UX disruption Feature:SLO obs-accessibilty A11Y issues in the Obs. solution. Project:Accessibility Team:obs-ux-management Observability Management User Experience Team WCAG A

Comments

@bhavyarm
Copy link
Contributor

**Kibana version:**QA observability main

Browser version: chrome latest

Describe the bug: Observability ->SLOs page -> Metric sparkline charts fail elements with an img role must have an alternative text a11y check.

This is the success criteria to meet this check

Element location:

.css-9sbomz-euiFlexItem-grow-1.euiFlexItem:nth-child(1) > .euiPanel--plain._StyledEuiPanel-sc-1hcbbka-0[title="VIOLATED"] > .echChart > .echChartContent > .echContainer > .echChartPointerContainer > .echMetricContainer[role="list"] > li > .echMetric[role="figure"] > .echSingleMetricSparkline > .echSingleMetricSparkline__svg[width="100%"][height="100%"]
Screenshot 2024-03-26 at 5 33 46 PM
@bhavyarm bhavyarm added bug Fixes for quality problems that affect the customer experience WCAG A Feature:SLO Team:obs-ux-management Observability Management User Experience Team defect-level-1 Critical UX disruption labels Mar 26, 2024
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-accessibility (Project:Accessibility)

@alexwizp alexwizp self-assigned this May 9, 2024
alexwizp added a commit to alexwizp/kibana that referenced this issue May 9, 2024
@alexwizp
fix: [AXE-CORE][Serverless - Observability] Observability -> SLOs -> …
fed8829 
…Metric sparkline charts fail <svg> elements with an img role must have an alternative text a11y check

Closes: elastic#179491
@alexwizp alexwizp mentioned this issue May 9, 2024
Merged
alexwizp added a commit that referenced this issue May 10, 2024
@alexwizp
fix: [AXE-CORE][Serverless - Observability] Observability -> SLOs -> …
00ceb7c 
…Metric sparkline charts fail <svg> elements with an img role must have an alternative text a11y check (#183040)

Closes: #179491

## Summary

Observability ->SLOs page -> Metric sparkline charts fail <svg> elements
with an img role must have an alternative text a11y check.

## What was done:? 
1. The `trendA11yTitle` attribute has been added within the
`SloCardChart` component to address accessibility concerns

## Screen: 

### AXE report: 

![image](https://github.com/elastic/kibana/assets/20072247/e366fcf8-fe4b-40a8-baf3-366003b3d16c)

### A11Y tree:

![image](https://github.com/elastic/kibana/assets/20072247/d2633990-a089-477c-8fe1-2d8fefadc8fd)
@smith smith added the obs-accessibilty A11Y issues in the Obs. solution. label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexwizp alexwizp

Labels
bug Fixes for quality problems that affect the customer experience defect-level-1 Critical UX disruption Feature:SLO obs-accessibilty A11Y issues in the Obs. solution. Project:Accessibility Team:obs-ux-management Observability Management User Experience Team WCAG A
Projects
None yet
Milestone
No milestone
4 participants
@smith @bhavyarm @elasticmachine @alexwizp