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

Sankey: chart broken when updating point with the same 'from' and 'to' #16080

Closed
crohit7 opened this issue Jul 20, 2021 · 5 comments · Fixed by #16082
Closed

Sankey: chart broken when updating point with the same 'from' and 'to' #16080

crohit7 opened this issue Jul 20, 2021 · 5 comments · Fixed by #16082
Assignees

Comments

@crohit7
Copy link

crohit7 commented Jul 20, 2021

Expected behaviour

Chart should render with updated color

Actual behaviour

Chart is broken

Live demo with steps to reproduce

https://jsfiddle.net/frnmdL5x/

Product version

9.1.2

Affected browser(s)

@raf18seb
Copy link
Contributor

Hi @crohit7, thanks for reporting the issue.

This happens when updating any point when one of the points goes to itself.

Simplified demo: https://jsfiddle.net/BlackLabel/aomk7fsd/

@raf18seb raf18seb changed the title For sankey series, If I try to update the color of series then chart got broken Sankey: chart broken when updating point with the same 'from' and 'to' Jul 20, 2021
@crohit7
Copy link
Author

crohit7 commented Jul 20, 2021

yes, exactly @raf18seb

@crohit7
Copy link
Author

crohit7 commented Jul 22, 2021

@raf18seb @khlieng
Looks like issue is fixed at your end.
Can we get beta version so that we can verify at our end
We have product launch in next week, could you please let us know by when we will get actual fix for this?
Thanks!

@khlieng
Copy link
Member

khlieng commented Jul 22, 2021

The release is probably at least about 3 weeks away, heres a plugin you can use while waiting: https://jsfiddle.net/ek8hdj1o/2/

@crohit7
Copy link
Author

crohit7 commented Jul 22, 2021

@khlieng Thanks for the update

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

Successfully merging a pull request may close this issue.

4 participants