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

3d scatter draggable demo rotated by 180 deg. doesn't hover proper points #4552

Closed
KacperMadej opened this Issue Sep 9, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@KacperMadej
Contributor

KacperMadej commented Sep 9, 2015

Demo: http://www.highcharts.com/demo/3d-scatter-draggable
Steps:

  1. Rotate 180 deg. in horizontal axis
  2. Hover over points
  3. Hovered points are not the one under the cursor, but on the other side instead.
@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Sep 10, 2015

Collaborator

Looks like either the k-d-tree is not using the z dimension correctly, or the k-d-points are not translated to 2D before applying. Alternatively, we shouldn't use k-d-tree at all for 3d.

Collaborator

TorsteinHonsi commented Sep 10, 2015

Looks like either the k-d-tree is not using the z dimension correctly, or the k-d-points are not translated to 2D before applying. Alternatively, we shouldn't use k-d-tree at all for 3d.

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 8, 2016

Contributor

+1

Contributor

pawelfus commented Feb 8, 2016

+1

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Mar 30, 2016

Collaborator

Demo with the fix applied: http://jsfiddle.net/highcharts/wzfes2yj/1/

Note that you now have to hover the point directly to read the tooltip, because we disabled the k-d-tree. An alternative fix may be to fix the tree to use projected coordinates.

Collaborator

TorsteinHonsi commented Mar 30, 2016

Demo with the fix applied: http://jsfiddle.net/highcharts/wzfes2yj/1/

Note that you now have to hover the point directly to read the tooltip, because we disabled the k-d-tree. An alternative fix may be to fix the tree to use projected coordinates.

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