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

Not all of the legend items are exported #8627

Closed
undefined77 opened this Issue Jul 16, 2018 · 7 comments

Comments

Projects
None yet
4 participants
@undefined77

undefined77 commented Jul 16, 2018

Expected behaviour

All of the legend items should be exported

Actual behaviour

Not all of the legend items are exported

Live demo with steps to reproduce

http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/legend/navigation-enabled-false/
or here
https://cloud.highcharts.com/edit/171518

Product version

Highcharts version 6.1.1

Affected browser(s)

@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan Jul 17, 2018

Contributor

Hi @undefined77,
In your demo there are 30 points. In the exported file (png or pdf) I see also 30 elements, so could you elaborate which items are missing and which browser do you use?

chart

Chrome: 67.0.3396.99

Contributor

sebastianbochan commented Jul 17, 2018

Hi @undefined77,
In your demo there are 30 points. In the exported file (png or pdf) I see also 30 elements, so could you elaborate which items are missing and which browser do you use?

chart

Chrome: 67.0.3396.99

@undefined77

This comment has been minimized.

Show comment
Hide comment
@undefined77

undefined77 Jul 18, 2018

chart 5
Version 67.0.3396.99 (Official Build) (64-bit)

undefined77 commented Jul 18, 2018

chart 5
Version 67.0.3396.99 (Official Build) (64-bit)

@undefined77

This comment has been minimized.

Show comment
Hide comment
@undefined77

undefined77 commented Jul 18, 2018

2018-07-18_12-10-41

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Jul 18, 2018

Contributor

Possible to recreate using Windows 10 / Chrome 67.0.3396.99

Contributor

KacperMadej commented Jul 18, 2018

Possible to recreate using Windows 10 / Chrome 67.0.3396.99

@KacperMadej KacperMadej added the Bug label Jul 18, 2018

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Jul 19, 2018

Contributor

Internal note:
Don't we simply need to increase exporting.chartOptions.chart.height? Demo works fine on osx, so probably windows font-sizes cause this.

Contributor

pawelfus commented Jul 19, 2018

Internal note:
Don't we simply need to increase exporting.chartOptions.chart.height? Demo works fine on osx, so probably windows font-sizes cause this.

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Jul 23, 2018

Contributor

@undefined77

Please share where in options are you disabling navigation for export or are you reporting that the demo from API reference is not showing what it supposed to show (in that case please share API reference where the demo is posted). When legend.navigation is disabled for exporting then it's disabled, so I'm not sure what's the problem here. Demo: http://jsfiddle.net/BlackLabel/8tnr4m5v/

Contributor

KacperMadej commented Jul 23, 2018

@undefined77

Please share where in options are you disabling navigation for export or are you reporting that the demo from API reference is not showing what it supposed to show (in that case please share API reference where the demo is posted). When legend.navigation is disabled for exporting then it's disabled, so I'm not sure what's the problem here. Demo: http://jsfiddle.net/BlackLabel/8tnr4m5v/

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Jul 23, 2018

Contributor

Fixed demo to work on Windows, fixed link in API reference to look and work better. PR pending.

Contributor

KacperMadej commented Jul 23, 2018

Fixed demo to work on Windows, fixed link in API reference to look and work better. PR pending.

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