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

Treemaps with zero value fail #5563

Closed
TorsteinHonsi opened this Issue Aug 4, 2016 · 5 comments

Comments

Projects
None yet
2 participants
@TorsteinHonsi
Collaborator

TorsteinHonsi commented Aug 4, 2016

This is a regression with Highcharts 4.2.6.

Actual behaviour

JS error when adding treemap points with value 0.

Live demo with steps to reproduce

http://jsfiddle.net/highcharts/1hj84pts/

Affected browser(s)

All

@nikithaachhini

This comment has been minimized.

Show comment
Hide comment
@nikithaachhini

nikithaachhini Aug 5, 2016

Hi @TorsteinHonsi even I have the same issue. I see that you've fixed and closed it. Is it merged in the repo and updated? So that I can have the updated library in my project.

nikithaachhini commented Aug 5, 2016

Hi @TorsteinHonsi even I have the same issue. I see that you've fixed and closed it. Is it merged in the repo and updated? So that I can have the updated library in my project.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Aug 5, 2016

Collaborator

It is merged to the master in this repo, but if you're referring to the npm module, it is not yet published there. It will be part of the next maintenance release.

Collaborator

TorsteinHonsi commented Aug 5, 2016

It is merged to the master in this repo, but if you're referring to the npm module, it is not yet published there. It will be part of the next maintenance release.

@nikithaachhini

This comment has been minimized.

Show comment
Hide comment
@nikithaachhini

nikithaachhini Aug 5, 2016

Okay thank you. Once it is published then probably the cdn for high charts would also be updated.

nikithaachhini commented Aug 5, 2016

Okay thank you. Once it is published then probably the cdn for high charts would also be updated.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Aug 5, 2016

Collaborator
Collaborator

TorsteinHonsi commented Aug 5, 2016

@nikithaachhini

This comment has been minimized.

Show comment
Hide comment
@nikithaachhini

nikithaachhini Aug 5, 2016

Sure..will do. Thank you once again.

nikithaachhini commented Aug 5, 2016

Sure..will do. Thank you once again.

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