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

Boosted area displays beyond pane when yAxis height is decreased and min is set to some positive value. #8731

Closed
PaulDalek opened this Issue Aug 6, 2018 · 10 comments

Comments

Projects
None yet
6 participants
@PaulDalek
Contributor

PaulDalek commented Aug 6, 2018

Expected behaviour

Area series starts from the the min value of yAxis (just like line series).

Actual behaviour

Area series goes beyond the min value and displays outside the pane.

Live demo

http://jsfiddle.net/BlackLabel/w8otdhnp/ - line series, works
http://jsfiddle.net/BlackLabel/c9ek6pdb/ - area series, does not work

@vishalpherwani

This comment has been minimized.

Show comment
Hide comment
@vishalpherwani

vishalpherwani Aug 13, 2018

Hi Paul, any updates on when can we get this fixed? the boosted area displays beyond the pane when we try to define some min value, it breaks our fundamental requirement to use the area chart with any other chart for data segregation and comparison purposes. Is there any ETA as to when can this be fixed?
Thanks!

vishalpherwani commented Aug 13, 2018

Hi Paul, any updates on when can we get this fixed? the boosted area displays beyond the pane when we try to define some min value, it breaks our fundamental requirement to use the area chart with any other chart for data segregation and comparison purposes. Is there any ETA as to when can this be fixed?
Thanks!

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Aug 16, 2018

Contributor

Hi @vishalpherwani - no ETA at this moment.

Internal note:
Issue is related to #6594 (and PR: #6995).

Contributor

pawelfus commented Aug 16, 2018

Hi @vishalpherwani - no ETA at this moment.

Internal note:
Issue is related to #6594 (and PR: #6995).

@pawelfus pawelfus added the Bug label Aug 16, 2018

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Aug 22, 2018

Collaborator

@cvasseng Any insight?

Collaborator

TorsteinHonsi commented Aug 22, 2018

@cvasseng Any insight?

@cvasseng

This comment has been minimized.

Show comment
Hide comment
@cvasseng

cvasseng Aug 22, 2018

Contributor

The answer is likely in this block. I think minVal needs to be set to yAxis.min in there if it's not a range series or a stacked series.

Contributor

cvasseng commented Aug 22, 2018

The answer is likely in this block. I think minVal needs to be set to yAxis.min in there if it's not a range series or a stacked series.

@vishalpherwani

This comment has been minimized.

Show comment
Hide comment
@vishalpherwani

vishalpherwani commented Aug 29, 2018

@cvasseng @TorsteinHonsi : any update on the fix?

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Aug 30, 2018

Collaborator

Hi @vishalpherwani, we're working on a fix, expect it to be fixed pretty soon.

Collaborator

TorsteinHonsi commented Aug 30, 2018

Hi @vishalpherwani, we're working on a fix, expect it to be fixed pretty soon.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Aug 30, 2018

Collaborator

Demo with the fix applied: http://jsfiddle.net/highcharts/c9ek6pdb/14/

We're just preparing a release, and expect it ready for publishing tomorrow or early next week.

Collaborator

TorsteinHonsi commented Aug 30, 2018

Demo with the fix applied: http://jsfiddle.net/highcharts/c9ek6pdb/14/

We're just preparing a release, and expect it ready for publishing tomorrow or early next week.

@vishalpherwani

This comment has been minimized.

Show comment
Hide comment
@vishalpherwani

vishalpherwani Aug 31, 2018

Thank you @TorsteinHonsi ! please update once it is published.

vishalpherwani commented Aug 31, 2018

Thank you @TorsteinHonsi ! please update once it is published.

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej

KacperMadej Sep 3, 2018

Contributor

@vishalpherwani version 6.1.2 was published.
Demo with version 6.1.2: http://jsfiddle.net/BlackLabel/8po7m3ct/

Contributor

KacperMadej commented Sep 3, 2018

@vishalpherwani version 6.1.2 was published.
Demo with version 6.1.2: http://jsfiddle.net/BlackLabel/8po7m3ct/

@vishalpherwani

This comment has been minimized.

Show comment
Hide comment
@vishalpherwani

vishalpherwani commented Sep 3, 2018

Thank you @KacperMadej

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