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

Offline Export of Pie Charts not working in Firefox #6481

Closed
cmaguire1990 opened this Issue Mar 17, 2017 · 9 comments

Comments

Projects
None yet
3 participants
@cmaguire1990

cmaguire1990 commented Mar 17, 2017

Expected behaviour

When implementing the latest version of highcharts (5.0.9), I have created various Pie type charts. Upon choosing the export to PDF option, I expect the PDF to download and upon opening, the Pie chart should be displayed.

Actual behaviour

I can download the PDF document ok, however, upon opening the PDF, the Pie image is missing. Title and series data, however, is visible.

piechart export

Live demo with steps to reproduce

I have realised that this issue can also be replicated on an official offline export demo on the highcharts site.
If we navigate to:
http://jsfiddle.net/gh/get/jquery/1.11.0/highslide-software/highcharts.com/tree/master/samples/highcharts/exporting/offline-download-demo/

and if we proceed to change the chart type from "area" to "pie" and re-run the project, then we download and try to view the pdf...we can see that the title and series data loads fine, however, the actual pie chart does not display. I have only noticed this issue with downloading Pie charts into PDF on Firefox when using the latest (5.0.9) version of offline exporting.

Affected browser(s)

Firefox only

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 21, 2017

Collaborator

Thanks for reporting!

It's an issue with the svg2pdf.js library, hereby reported at yWorks/svg2pdf.js#22.

Collaborator

TorsteinHonsi commented Mar 21, 2017

Thanks for reporting!

It's an issue with the svg2pdf.js library, hereby reported at yWorks/svg2pdf.js#22.

@TorsteinHonsi TorsteinHonsi added the Bug label Mar 21, 2017

@cmaguire1990

This comment has been minimized.

Show comment
Hide comment
@cmaguire1990

cmaguire1990 Mar 29, 2017

Any update on this issue?

cmaguire1990 commented Mar 29, 2017

Any update on this issue?

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 29, 2017

Collaborator

We are looking into it with the yWorks team, see yWorks/svg2pdf.js#22.

Collaborator

TorsteinHonsi commented Mar 29, 2017

We are looking into it with the yWorks team, see yWorks/svg2pdf.js#22.

@cmaguire1990

This comment has been minimized.

Show comment
Hide comment
@cmaguire1990

cmaguire1990 Mar 29, 2017

Unfortunately, replacing the 2 files provided does not resolve the issue I am experiencing.
I updated both svg2pdf.js & svg2pdf.src.js within Highcharts-5.0.9\code\lib

cmaguire1990 commented Mar 29, 2017

Unfortunately, replacing the 2 files provided does not resolve the issue I am experiencing.
I updated both svg2pdf.js & svg2pdf.src.js within Highcharts-5.0.9\code\lib

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 31, 2017

Collaborator

It works nicely for me. Are you sure you're loading the correct files? Make sure the libURL setting is correct.

skjermbilde 2017-03-31 kl 10 36 40

Collaborator

TorsteinHonsi commented Mar 31, 2017

It works nicely for me. Are you sure you're loading the correct files? Make sure the libURL setting is correct.

skjermbilde 2017-03-31 kl 10 36 40

@cmaguire1990

This comment has been minimized.

Show comment
Hide comment
@cmaguire1990

cmaguire1990 Mar 31, 2017

libURL:"https://code.highcharts.com/5.0.9/lib/

Maybe the files haven't been updated on the highcharts site?

cmaguire1990 commented Mar 31, 2017

libURL:"https://code.highcharts.com/5.0.9/lib/

Maybe the files haven't been updated on the highcharts site?

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 31, 2017

Collaborator

Correct, v5.0.10 isn't published yet. It will probably be today, we're almost finished regression testing.

Collaborator

TorsteinHonsi commented Mar 31, 2017

Correct, v5.0.10 isn't published yet. It will probably be today, we're almost finished regression testing.

@cmaguire1990

This comment has been minimized.

Show comment
Hide comment
@cmaguire1990

cmaguire1990 commented Mar 31, 2017

Thank you

@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan Apr 10, 2017

Contributor

Works since HC 5.0.10.

Tested in FF 52.x.

Contributor

sebastianbochan commented Apr 10, 2017

Works since HC 5.0.10.

Tested in FF 52.x.

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