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

Can not remove chart with areasplinerange series while hovering #20722

Closed
KaisZaoualiWincher opened this issue Feb 27, 2024 · 3 comments
Closed

Comments

@KaisZaoualiWincher
Copy link

KaisZaoualiWincher commented Feb 27, 2024

Expected behaviour

Removing chart with areasplinerange series while hovering should be performed with no issues

Actual behaviour

Removing chart with areasplinerange series while hovering is throwing error: Cannot read properties of undefined (reading 'chart')

image

Live demo with steps to reproduce

https://codesandbox.io/p/sandbox/highcharts-99yys8

While hovering on the chart series, you need to press any key (which should hide the chart). Make sure you are selecting the preview (i.e. you can inspect the rendered chart to make sure the pressed key will have an impact on the chart)

Product version

"highcharts": "11.3.0",
"highcharts-react-official": "3.2.1"

Affected browser(s)

Google chrome

@hubertkozik
Copy link
Member

hubertkozik commented Feb 28, 2024

Hi @KaisZaoualiWincher!
I've just tested the chart config you provided and I cannot see any issues. I've changed the state to hide and show the chart on key press. I'm attaching a GIF below. Can you double check?
Live demo: https://codesandbox.io/p/sandbox/highcharts-forked-x6v7s3
not_a_bug

@PinkChampagne17
Copy link

Hi @hubertkozik!

I encountered the same issue and created a minimal reproduction, which should have a higher reproduction rate.

Reproduction steps:

  1. Open the live demo https://codesandbox.io/p/sandbox/highcharts-forked-cns4kh
  2. Open the preview in a new tab
  3. Mouse over the chart and don't move out
  4. Press any key to destroy the instance and the issue will be reproduced

I've tried as much as possible to enhance the reproducibility of the problem, but still can't achieve 100% reproducibility. If not reproduce it successfully, please try a few more times.

@hubertkozik
Copy link
Member

Thanks @PinkChampagne17!
I've reproduced and recreated the demo in vanilla JS: https://jsfiddle.net/BlackLabel/ghe9rnbx/

This issue is already reported here: #20572 You can track the progress there. I am closing this one as a duplicate.

@hubertkozik hubertkozik closed this as not planned Won't fix, can't repro, duplicate, stale Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants