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

[Highcharts Accessibility] For area chart, keyboard navigation (tab & arrow key) doesn't set focus on chart series at all. #7423

Closed
SummerLi opened this issue Nov 18, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@SummerLi
Copy link

@SummerLi SummerLi commented Nov 18, 2017

Expected behaviour

For area chart, keyboard navigation (tab & arrow key) should work.

Actual behaviour

For area chart, keyboard navigation (tab & arrow key) doesn't set focus on chart series at all.

Live demo with steps to reproduce

Affected browser(s)

Edge, Chrome

@SummerLi
Copy link
Author

@SummerLi SummerLi commented Nov 18, 2017

@oysteinmoseng oysteinmoseng self-assigned this Nov 20, 2017
@oysteinmoseng
Copy link
Member

@oysteinmoseng oysteinmoseng commented Nov 20, 2017

Seems to work in latest code, see: http://jsfiddle.net/oysteinmoseng/bp81738j/1/show/.

@SummerLi Can you verify that this works for you? The latest code can be found at http://github.highcharts.com/bugfix/modules/accessibility.js

@SummerLi
Copy link
Author

@SummerLi SummerLi commented Nov 20, 2017

Accessibility_v6.0.2.zip
@oysteinmoseng I can still repro this with the latest accessibility.js.

I've attached above demo zip file, the area chart script is: Highcharts_Demo/Highcharts_Demo/Accessibility_v6.0.2/area-basic/index.htm

@oysteinmoseng
Copy link
Member

@oysteinmoseng oysteinmoseng commented Nov 28, 2017

This was not an issue with area series specifically, but an issue with the default behavior for skipNullPoints (now enabled by default) and pointDescriptionThreshold (now disabled by default, and more predictable behavior for keyboard navigation when enabled).

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

Successfully merging a pull request may close this issue.

None yet
2 participants