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

Treegraph leaves unnecessary links after series.setData #19524

Closed
artzub opened this issue Aug 9, 2023 · 1 comment · Fixed by #19625
Closed

Treegraph leaves unnecessary links after series.setData #19524

artzub opened this issue Aug 9, 2023 · 1 comment · Fixed by #19625

Comments

@artzub
Copy link

artzub commented Aug 9, 2023

Hello!

I met the problem that if we dynamically set data for a treegraph series by series.setData method, the chart leaves links that must be deleted, or they are links between unexisting nodes.
If we do that by series.update({ data }) all works fine but links lost animation.

Expected behaviour

The chart leaves only links between existing nodes if data is updated by series.setData method.
Or another way if data is updated by series.update method, keep links animation.

Actual behaviour

After updating data by series.setData method, the chart leaves unnecessary links. And after some updates throw error:

highstock.src.js:45857 
Uncaught TypeError: Cannot read properties of undefined (reading 'y')
    at y.g (highstock.src.js:45857:1)
    at highstock.src.js:1748:1
    at Array.forEach (<anonymous>)
    at f (highstock.src.js:1745:1)
    at y.generatePoints (highstock.src.js:32552:1)
    at y.t (highstock.src.js:47125:1)
    at y.<anonymous> (highstock.src.js:914:1)
    at y.h (boost.src.js:2976:1)
    at b.<computed> (highstock.src.js:912:1)
    at y.translate (highstock.src.js:32693:1)

Live demo with steps to reproduce

Reproducing the bug

image
image

Product version

Highcharts: 11.1.0

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Aug 9, 2023
@jakubjanuchta jakubjanuchta self-assigned this Aug 10, 2023
@bm64
Copy link
Member

bm64 commented Aug 11, 2023

Hi @artzub and thanks for reporting the issue.

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