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

Line is missing after zoom #7061

Closed
sebastianbochan opened this issue Aug 16, 2017 · 3 comments
Closed

Line is missing after zoom #7061

sebastianbochan opened this issue Aug 16, 2017 · 3 comments

Comments

@sebastianbochan
Copy link
Contributor

@sebastianbochan sebastianbochan commented Aug 16, 2017

Expected behaviour

Line should be visible

Actual behaviour

Line is missing, when zooming between points.

Live demo with steps to reproduce

Demo:

zooming

Affected browser(s)

All

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented Aug 16, 2017

Regression after a6c5ecf

@TorsteinHonsi TorsteinHonsi self-assigned this Aug 17, 2017
@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Aug 17, 2017

In my opinion, #6203 was a mistake. The extremes should take into account the first point outside the visible range just for this reason. I'll revert the fix and write a test for it.

@sebastianbochan
Copy link
Contributor Author

@sebastianbochan sebastianbochan commented Aug 17, 2017

I agree with that, it is a reason of 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
3 participants