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

null point data when clicking on point with drilldown #7005

Closed
alvarezguille opened this issue Jul 25, 2017 · 4 comments
Closed

null point data when clicking on point with drilldown #7005

alvarezguille opened this issue Jul 25, 2017 · 4 comments
Labels
Status: Stale This issue hasn't had any activity for a while, and will be auto-closed if no further updates occur. Type: Bug

Comments

@alvarezguille
Copy link
Contributor

Looks like a regression of #3383

Expected behaviour

Point data is not null

Live demo with steps to reproduce

copied from #3383

No drilldown: http://jsfiddle.net/2ho4wewo/
Drilldown: http://jsfiddle.net/97nma53g/

@KacperMadej
Copy link

Hi,

Thank you for reporting about the problem. I am unable to find a version in which the problem wasn't visible, so until that code version is found this is not a regression - if you have any more info, please share.

You could use drilldown event of a chart to get the point before it's destroyed.
API reference: http://api.highcharts.com/highcharts/chart.events.drilldown
Demo: http://jsfiddle.net/97nma53g/2/

@alvarezguille
Copy link
Contributor Author

Thanks for the quick answer!
I based my thought on the comment on the original issue stating that it was fixed before closing the issue

Fixed in the master branch

Regression or not I think it's still not the expected behavior.

@TorsteinHonsi
Copy link
Collaborator

The point is already destroyed at the time the click event triggers, so we can't access its properties. This happens before the drilldown module's click handler is added before the handler that's added in the options.

If we decide to fix this, we need to change the order so the drilldown happens last, and preferably be able to cancel the drilldown from the click event.

Workaround

You can use the chart.events.drilldown event to access the point before it is destroyed: http://jsfiddle.net/97nma53g/3/

@stale
Copy link

stale bot commented Sep 24, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions!

@stale stale bot added the Status: Stale This issue hasn't had any activity for a while, and will be auto-closed if no further updates occur. label Sep 24, 2019
@stale stale bot closed this as completed Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Stale This issue hasn't had any activity for a while, and will be auto-closed if no further updates occur. Type: Bug
Projects
None yet
Development

No branches or pull requests

3 participants