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

SVG export error #16589

Closed
webhazrat opened this issue Nov 4, 2021 · 6 comments · Fixed by #16766
Closed

SVG export error #16589

webhazrat opened this issue Nov 4, 2021 · 6 comments · Fixed by #16766
Assignees
Labels

Comments

@webhazrat
Copy link

When export this chart by svg format an error occur.
Error Like:
This page contains the following errors:
error on line 1 at column 284: Extra content at the end of the document
Below is a rendering of the page up to the first error.

@highsoft-bot highsoft-bot added this to To do in Development-Flow Nov 4, 2021
@raf18seb
Copy link
Contributor

raf18seb commented Nov 4, 2021

Hi @webhazrat, could you provide more info? I think you forgot to attach something to your post.

@weathda
Copy link

weathda commented Dec 1, 2021

We have also experienced this issue with download SVG images from Highchart (v9.1.2).

image

Opening the image with Edge or Chrome results in the same error.

image

@weathda
Copy link

weathda commented Dec 1, 2021

Retested with 9.3.2 and still seeing the issue. Attached is the chart file. Remove .txt to get the original svg file.

chart.svg.txt

@raf18seb
Copy link
Contributor

raf18seb commented Dec 6, 2021

Hi @weathda, thank you for providing more info.

I was able to reproduce the issue with an accessibility.js module.
Demo: https://jsfiddle.net/BlackLabel/y2Lpo3jr/

Seems like it's caused by the proxy elements added by this commit: c2d2912

However, I wasn't able to reproduce the issue in v9.1.2.
It's working fine in v9.2.2, broken since v9.3.0 (current version is v9.3.2), but maybe I've missed something?

@weathda
Copy link

weathda commented Dec 7, 2021

Oops, looks like my older svg was generated with 9.3.2 so I guess I had a newer version.

Thanks, I'll look for the fix in a newer release.

@oysteinmoseng
Copy link
Member

Here is a drop-in workaround for this issue for those of you waiting for the next release: https://jsfiddle.net/oysteinmoseng/yezu6nx3/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging a pull request may close this issue.

5 participants