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

SpiderWeb arearange and areasplinerange fill discontinuity #6184

Closed
cromulus opened this Issue Dec 28, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@cromulus

cromulus commented Dec 28, 2016

Expected behaviour

The area range spline should wrap around the spider web chart consistently, connecting the high and low value of each data point on in the range series.

Actual behaviour

the step between the last element in the series and the first element in the series on the spiderweb chart data set, the spline seems to somehow result in a negative value.

The behavior varies between different browsers, which leads me to believe this is not the expected output.

Note: the lines seem to function as expected, it is the area fill that seems to break.

This also breaks with 'arearange' as well.

Live demo with steps to reproduce

http://jsfiddle.net/cromulus/v0r9dtcm/

Affected browser(s)

Chrome: 55.0.2883.95 (64-bit)
Safari: 10.0.2 (12602.3.12.0.1)

@cromulus cromulus changed the title from SpiderWeb areasplinerange discontinuity to SpiderWeb arearange and areasplinerange fill discontinuity Dec 28, 2016

@cromulus

This comment has been minimized.

Show comment
Hide comment
@cromulus

cromulus Dec 28, 2016

It appears that the same issue is effecting polar arearange charts as well:
http://jsfiddle.net/highcharts/30q95we5/
http://jsfiddle.net/highcharts/30q95we5/

cromulus commented Dec 28, 2016

It appears that the same issue is effecting polar arearange charts as well:
http://jsfiddle.net/highcharts/30q95we5/
http://jsfiddle.net/highcharts/30q95we5/

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Dec 29, 2016

Contributor

Thank you for reporting this issue!

It used to work in 4.2.1: http://jsfiddle.net/v0r9dtcm/7/

Related issue: #5528

Contributor

pawelfus commented Dec 29, 2016

Thank you for reporting this issue!

It used to work in 4.2.1: http://jsfiddle.net/v0r9dtcm/7/

Related issue: #5528

@pawelfus pawelfus added the Regression label Dec 29, 2016

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Jan 5, 2017

Collaborator

It works as expected once we set connectEnds to true: http://jsfiddle.net/v0r9dtcm/8/

Collaborator

TorsteinHonsi commented Jan 5, 2017

It works as expected once we set connectEnds to true: http://jsfiddle.net/v0r9dtcm/8/

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