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

Drilldown.animation = false causes Uncaught RangeError: Maximum call stack size exceeded #19373

Closed
CoryKniefel opened this issue Jul 14, 2023 · 3 comments · Fixed by #19472
Closed

Comments

@CoryKniefel
Copy link

CoryKniefel commented Jul 14, 2023

Expected behaviour

When setting the drilldown.animation to 'false', the chart should function as expected when user attempts to drilldown. As described: https://api.highcharts.com/highmaps/drilldown

Actual behaviour

The chart hangs for a few seconds and then an error is logged to console: Uncaught RangeError: Maximum call stack size exceeded

Live demo with steps to reproduce

https://jsfiddle.net/f1wv8ys3/6/

Notice line 139.

  • Open console
  • Click on a state in the chart
  • Wait for error in console
  • Change line 139 to be true
  • Save/run
  • Click on a state in the chart
  • Observe glorious drill-down

Product version

I'm seeing this in 11.1.0, the only version I've tried.

Affected browser(s)

All that I've tried, which is just Edge.

@highsoft-bot highsoft-bot added this to To do in Development-Flow via automation Jul 14, 2023
@bm64
Copy link
Member

bm64 commented Jul 17, 2023

Hello @CoryKniefel and thanks for reporting the issue.

Seems to work fine in 11.0.0 and broken since v11.0.1.

Internal note
Bisected to: e07b3f9 FYI @hubertkozik

@CoryKniefel
Copy link
Author

I updated the details to say I only tried in 11.1.0. I had tried other versions, but not for this issue.

@hubertkozik
Copy link
Member

hubertkozik commented Jul 27, 2023

Hello guys!

I have prepared a workaround for that issue. It is inside the IIFE function in the below demo. If you want to use it, just copy-paste it into your project and everything should work as intended. A similar solution will be used in the Drilldown module code.

Live demo: https://jsfiddle.net/BlackLabel/v0a96f8L/

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