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

Annotations are displaying incorrectly after zooming a chart. #7826

Closed
Denyllon opened this issue Feb 8, 2018 · 3 comments
Closed

Annotations are displaying incorrectly after zooming a chart. #7826

Denyllon opened this issue Feb 8, 2018 · 3 comments

Comments

@Denyllon
Copy link
Contributor

@Denyllon Denyllon commented Feb 8, 2018

Expected behaviour

Annotations which aren't in current selection range should be invisible.

Actual behaviour

Annotations which should be invisible after zoom to certain selection, instead they are moved to the edge of plotArea (shapes on the left side, labels on the right).

It occurs only when there is a big amount of data points.

Live demo with steps to reproduce

http://jsfiddle.net/BlackLabel/jzqdmvwt/

  1. Zoom chart.

Occurence

v6.0.4 and below
From v6.0.5 all annotations are hidden.

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Feb 9, 2018

The problem with wrong display out of visible range was resolved in 6.0.5 version.

About current (6.0.6 version):
Markers needs to be enabled to show the annotations in the current (6.0.6) version.
http://jsfiddle.net/BlackLabel/jzqdmvwt/1/

Is the limitation to an existing marker intentional? If yes, docs should be updated. If no, then this is a bug. @morganfree ?

@morganfree
Copy link
Contributor

@morganfree morganfree commented Feb 9, 2018

@KacperMadej, it's a regression bug. I will fix it.

@manometer114
Copy link

@manometer114 manometer114 commented Feb 15, 2018

Thank you everybody for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants