Skip to content

Docs highcharts eng a11y #4335

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

Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

Docs highcharts eng a11y #4335

wants to merge 74 commits into from

Conversation

krisantrobus
Copy link
Collaborator

  • New Accessibility Page: Added a dedicated page, accessibility.mdx, under the data visualization section to explain Highcharts' accessibility features, including keyboard navigation, screen reader support, and customization options. The page includes examples and guidelines for implementing accessible charts.

  • Chart Options Update: Enhanced the BaseChart options in baseFormatter.ts to include an accessibility property, allowing users to configure accessibility features directly in the chart options.

Co-authored-by: Sarah <sali@twilio.com>
krisantrobus and others added 2 commits June 17, 2025 07:13
…n/for-engineers/accessibility.mdx

Co-authored-by: Sarah <sali@twilio.com>
serifluous
serifluous previously approved these changes Jun 17, 2025
PixeledCode
PixeledCode previously approved these changes Jun 18, 2025
Base automatically changed from chart-types-line-column-support to main June 18, 2025 15:58
@kodiakhq kodiakhq bot dismissed stale reviews from PixeledCode, serifluous, and nkrantz June 18, 2025 15:58

The base branch was changed.

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 18, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Jun 18, 2025
@krisantrobus krisantrobus added the 🕵🏻‍♀️ Run website visual regression When applied, we will run a full suite of visual regression tests across the doc site label Jun 18, 2025
Copy link

cypress bot commented Jun 18, 2025

Paste    Run #9879

Run Properties:  status check passed Passed #9879  •  git commit 0e26a2c382 ℹ️: Merge 5da57de39bea393f36d869886bb31a887000ae1f into 3705495c77495f732db1c4796c61...
Project Paste
Branch Review docs-highcharts-eng-a11y
Run status status check passed Passed #9879
Run duration 07m 38s
Commit git commit 0e26a2c382 ℹ️: Merge 5da57de39bea393f36d869886bb31a887000ae1f into 3705495c77495f732db1c4796c61...
Committer krisantrobus
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 125
View all changes introduced in this branch ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Doc Site Related to the documentation website Area: Repo lgtm This PR has been approved by a maintainer 🕵🏻‍♀️ Run website visual regression When applied, we will run a full suite of visual regression tests across the doc site size:L This PR changes 100-499 lines, ignoring generated files. Type: Documentation Improvements or additions to documentation Type: Tests Adds tests to the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants