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

Jump to bottom

Print chart in the full screen mode - console error #16126

Closed
Basts opened this issue Aug 2, 2021 · 0 comments · Fixed by #16128
Closed

Print chart in the full screen mode - console error #16126

Basts opened this issue Aug 2, 2021 · 0 comments · Fixed by #16128
Assignees
@khlieng
Labels
a11y Accessibility related. Flow: Done status Type: Bug
Projects
Development-Flow

Comments

@Basts
Copy link
Contributor

Basts commented Aug 2, 2021

The error occurs in the console after triggering the print event while the full-screen mode is applied. As a consequence full-screen mode is forced to be disabled but in the export menu the option to close still exists.

accessibility.src.js:431 Uncaught TypeError: a.hasAttribute is not a function

Live demo with steps to reproduce

Demo: https://www.highcharts.com/demo/line-basic

Steps to reproduce:

  1. Open export menu,
  2. Click View in full screen,
  3. Open export menu again,
  4. Click Print chart,
  5. Check the console and export menu.
@circleci-mu circleci-mu added this to To do in Development-Flow via automation Aug 2, 2021
@khlieng khlieng added a11y Accessibility related. Type: Bug labels Aug 2, 2021
@khlieng khlieng self-assigned this Aug 2, 2021
khlieng added a commit that referenced this issue Aug 2, 2021
@khlieng
Fixed #16126, printing in full screen with a11y loaded threw
157785d
@khlieng khlieng mentioned this issue Aug 2, 2021
Merged
@circleci-mu circleci-mu moved this from To do to Review in progress in Development-Flow Aug 2, 2021
Development-Flow automation moved this from Review in progress to Done Aug 13, 2021
TorsteinHonsi pushed a commit that referenced this issue Aug 13, 2021
@khlieng @TorsteinHonsi
Fixed #16126, printing in full screen with a11y loaded threw
b7b6b71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@khlieng khlieng

Labels
a11y Accessibility related. Flow: Done status Type: Bug
Projects
Development-Flow
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bugfix/16126-a11y-full-screen-print-error highcharts/highcharts
3 participants
@khlieng @Basts @circleci-mu