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

Spiderweb/Windrose/Radar charts not displaying all x labels #13762

Closed
IgorFIE opened this issue Jun 23, 2020 · 3 comments · Fixed by #13922
Closed

Spiderweb/Windrose/Radar charts not displaying all x labels #13762

IgorFIE opened this issue Jun 23, 2020 · 3 comments · Fixed by #13922
Assignees

Comments

@IgorFIE
Copy link

IgorFIE commented Jun 23, 2020

Expected behaviour

Given: I have a spiderweb chart
When: I enable useHTML on the chart labels
Then: I should see all labels on the spiderweb chart

Actual behaviour

It is only drawing half of the labels.
This also happens in windrose and Polar (radar) chart

Live demo with steps to reproduce

https://jsfiddle.net/panintelligence/kqt0r9w1/

Product version

We updated from 7.2.0 to 8.1.0 and the problem started to appear.
This is also a problem in the last version

@pawelfus
Copy link
Contributor

Thank you for reporting the issue @IgorFIE

Broken since v8.1.0, up to v8.0.4 it was fine.

Internal note:
Bisect fails. Just a guess, but might be connected: 94a2263 - @PaulDalek could you check it?

@PaulDalek
Copy link
Contributor

Hi @IgorFIE,

In a nutshell, the xOffset variable used for calculating a label's x position is NaN when useHTML is set to true. As for now you can use a simple workaround which prevents NaN in calculation.

Workaround

https://jsfiddle.net/BlackLabel/zmah2swf/

@IgorFIE
Copy link
Author

IgorFIE commented Jun 24, 2020

Hi @pawelfus and @PaulDalek ,

For now we decided to downgrade to 8.0.4 until there's a version with the fix.
I'll keep an eye on this issue.

Thanks.

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

Successfully merging a pull request may close this issue.

4 participants