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.

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

fix(xy): line annotation marker aria label fix #2558

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

rbrtj
Copy link
Contributor

@rbrtj rbrtj commented Oct 25, 2024

Summary

Fix for #2 in #2520

Details

Added an aria-label to the clickable annotation button marker. No Axe errors are visible after the changes.
image

Issues

#2520

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)

@nickofthyme
Copy link
Collaborator

Sorry for the delay, we've been tied up elsewhere. I'll get around to reviewing soon.

Copy link
Collaborator

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @rbrtj I should have just approved this a while ago. Sorry for the delay of such a simple fix.

@nickofthyme nickofthyme added :annotation Annotation (line, rect, text) related issue :accessibility Accessibility related issue :xy Bar/Line/Area chart related labels Nov 14, 2024
@nickofthyme nickofthyme enabled auto-merge (squash) November 14, 2024 16:04
@nickofthyme
Copy link
Collaborator

buildkite test this

@nickofthyme nickofthyme merged commit 3b056c3 into elastic:main Nov 14, 2024
14 checks passed
nickofthyme pushed a commit that referenced this pull request Nov 28, 2024
## [68.0.3](v68.0.2...v68.0.3) (2024-11-28)

### Bug Fixes

* **xy:** line annotation marker aria label fix ([#2558](#2558)) ([3b056c3](3b056c3))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:accessibility Accessibility related issue :annotation Annotation (line, rect, text) related issue :xy Bar/Line/Area chart related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants