-
Notifications
You must be signed in to change notification settings - Fork 117
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
Allow pasing labelling props to <canvas>
element
#1096
Labels
:accessibility
Accessibility related issue
bug
Something isn't working
released
Issue released publicly
Comments
myasonik
added
bug
Something isn't working
:accessibility
Accessibility related issue
labels
Mar 30, 2021
2 tasks
rshen91
added a commit
that referenced
this issue
Apr 22, 2021
Fixes #1096 BREAKING CHANGE: `description` prop in `<Settings/>` is renamed to `ariaDescription` Co-authored-by: Marco Vettorello <vettorello.marco@gmail.com>
nickofthyme
pushed a commit
that referenced
this issue
Apr 22, 2021
# [29.0.0](v28.2.0...v29.0.0) (2021-04-22) ### Features * **a11y:** add label for screen readers ([#1121](#1121)) ([920e585](920e585)), closes [#1096](#1096) * **annotations:** marker body with dynamic positioning ([#1116](#1116)) ([601abac](601abac)) ### BREAKING CHANGES * **a11y:** `description` prop in `<Settings/>` is renamed to `ariaDescription` Co-authored-by: Marco Vettorello <vettorello.marco@gmail.com>
🎉 This issue has been resolved in version 29.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
AMoo-Miki
pushed a commit
to AMoo-Miki/OpenSearch-Dashboards
that referenced
this issue
Feb 10, 2022
# [29.0.0](elastic/elastic-charts@v28.2.0...v29.0.0) (2021-04-22) ### Features * **a11y:** add label for screen readers ([opensearch-project#1121](elastic/elastic-charts#1121)) ([ddb8782](elastic/elastic-charts@ddb8782)), closes [opensearch-project#1096](elastic/elastic-charts#1096) * **annotations:** marker body with dynamic positioning ([#1116](elastic/elastic-charts#1116)) ([997d487](elastic/elastic-charts@997d487)) ### BREAKING CHANGES * **a11y:** `description` prop in `<Settings/>` is renamed to `ariaDescription` Co-authored-by: Marco Vettorello <vettorello.marco@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
:accessibility
Accessibility related issue
bug
Something isn't working
released
Issue released publicly
Describe the bug
There is no way to set
aria-label
oraria-labelledby
on a chart.Notes on implementation
aria-*
props directly because in the long-term we won't use many. Rather, I'd recommend usinglabel
andlabelledby
respectively. This, I hope, is both clear as a standalone prop name and hopefully hints at what their implementation might be/mean.The text was updated successfully, but these errors were encountered: