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

Boost: Issue with ConnectNulls when zooming #7730

Closed
NeuwenWolf opened this Issue Jan 25, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@NeuwenWolf

NeuwenWolf commented Jan 25, 2018

Expected behaviour

Null points should never be connected.

Actual behaviour

In specific circumstances (which I cannot fully pin down), points that are not within the zoomed range (i.e. not visible) are connected.

Live demo with steps to reproduce

http://jsfiddle.net/vd1y2cjx/7/
Zoom in on the two points in Series 2 to see Series 1 connect
image
image
image

Product version

Development version of boost.js
Latest stable of highcharts.js

Affected browser(s)

Only tested in Chrome

Further Comments

It seems you need at least 3 series to make it happen.
You don't need values on either side, here is the edge of a chart where there are no values to the left, but the chart is drawing horizontal null-value lines:
image

@sebastianbochan

This comment has been minimized.

Show comment
Hide comment
@sebastianbochan

sebastianbochan Jan 26, 2018

Contributor

Hi @NeuwenWolf
Thank you for reporting

Internal note
Looks like similiar issue like #7716

@cvasseng any comments?

Contributor

sebastianbochan commented Jan 26, 2018

Hi @NeuwenWolf
Thank you for reporting

Internal note
Looks like similiar issue like #7716

@cvasseng any comments?

@cvasseng

This comment has been minimized.

Show comment
Hide comment
@cvasseng

cvasseng Jan 30, 2018

Contributor

A fix has been pushed, and is pending merge.

Here's a preview: http://jsfiddle.net/baz8mfLh/

Contributor

cvasseng commented Jan 30, 2018

A fix has been pushed, and is pending merge.

Here's a preview: http://jsfiddle.net/baz8mfLh/

@cvasseng

This comment has been minimized.

Show comment
Hide comment
@cvasseng

cvasseng Feb 2, 2018

Contributor

Fixed in 6.0.5.

Contributor

cvasseng commented Feb 2, 2018

Fixed in 6.0.5.

@cvasseng cvasseng closed this Feb 2, 2018

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