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

Highstock click on zoomed area leaves two columns selected #6445

Closed
ShMcK opened this issue Mar 9, 2017 · 5 comments
Closed

Highstock click on zoomed area leaves two columns selected #6445

ShMcK opened this issue Mar 9, 2017 · 5 comments

Comments

@ShMcK
Copy link

ShMcK commented Mar 9, 2017

Using Highstock.

Steps to Reproduce

  1. Click on a column bar on the far left.
  2. Zoom in on the far right.
  3. Click on a bar within the zoomed in area.
  4. Zoom out.

Expected behaviour

One column to be highlighted.

Actual behaviour

The original column remains highlighted as well as selected bar, leaving two items selected.

Also the zoom animation is not performing as expected.

Example

chart issue

Affected browser(s)

All

@jon-a-nygaard
Copy link
Contributor

Internal Note: Simplified demo

@TorsteinHonsi
Copy link
Collaborator

The cause of the problem is that the series points array is cropped to the current viewing range. When running point.select, the method reads getSelectedPoints in order to unselect all other points, but this doesn't return the points that are outside the viewing range.

Workaround

Set the cropThreshold higher and disable data grouping: http://jsfiddle.net/pnewwpfp/3/

@ShMcK
Copy link
Author

ShMcK commented Mar 14, 2017

Thank you, the workaround worked great.

@KacperMadej KacperMadej self-assigned this Mar 24, 2017
KacperMadej pushed a commit that referenced this issue Mar 27, 2017
TorsteinHonsi pushed a commit that referenced this issue Apr 3, 2017
# Conflicts:
#	samples/unit-tests/point/point/demo.js
@KacperMadej
Copy link

@TorsteinHonsi Only the test is added and the commit with the fix is not. Is there something wrong with the suggested fix?

@TorsteinHonsi
Copy link
Collaborator

Thanks for notifiying, it seems I didn't notice the fix itself.

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

4 participants