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

Added Accessibility Support for Screenreaders #4936

Open
wants to merge 6 commits into
base: master
from

Conversation

@ish-joshi
Copy link

@ish-joshi ish-joshi commented Jul 10, 2020

PR Checklist:

  • I have tested this extensively and it does not break any existing behavior.
  • I have added/updated examples and tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

The PR adds support for screen-readers. The chart can now be detected and a description is generated dynamically for bar, line and pie charts common use case scenarios.

Previously, the charts were just ignored by the screen-reader. This is a major improvement and paves the way forward for better accessibility implementation for this amazing library.

Adds autogenerated description for screenreader as well as the ability for the user to add accessibility description which will be read out when the screen-reader focuses on the chart.

chart.setAccessibilitySummaryDescription("This is a customizable description on user end");

Accessibility is really important, 2.2 billion users have some sort of vision-impairment according to the WHO. This enables access for those sets of users who cannot see and makes the library charts accessible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.