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

Console errors "<text> attribute x: Expected length, "NaN"" when plotBands.label.text is set, and toggling visibility of series. #7175

Closed
Denyllon opened this Issue Sep 21, 2017 · 11 comments

Comments

Projects
None yet
6 participants
@Denyllon
Contributor

Denyllon commented Sep 21, 2017

Expected behaviour

Clean console, and label.text should be hidden after click on series in legend.

Actual behaviour

  1. There are lot of errors in console: attribute x: Expected length, "NaN"
  2. Label of plotBand not disappear with series, only move to the upper left corner of chart.

Everything works correctly upper to version 5.0.9.

Live demo with steps to reproduce

JSFiddle

  1. Open console in Developer Tools.
  2. Click on "Series 1" in legend.

Affected browser(s)

All browsers

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Sep 21, 2017

Contributor

Hi @Denyllon

Thanks for reporting this issue!

Internal note:
It's a regression after 2bf1b0c

Contributor

pawelfus commented Sep 21, 2017

Hi @Denyllon

Thanks for reporting this issue!

Internal note:
It's a regression after 2bf1b0c

@atshaas

This comment has been minimized.

Show comment
Hide comment
@atshaas

atshaas Oct 8, 2017

Could this still be an issue?
I can see similar recursive error message.
highcharts.min.js:61 Error: <rect> attribute y: Expected length, "NaNNaN". highcharts.min.js:61 Error: <rect> attribute height: Expected length, "NaNNaN".
Currently using chart.yAxis[0].removePlotLine('plotLine') and chart.xAxis[0].addPlotLine(options). I will debug and make a proper report when it remains a problem.

atshaas commented Oct 8, 2017

Could this still be an issue?
I can see similar recursive error message.
highcharts.min.js:61 Error: <rect> attribute y: Expected length, "NaNNaN". highcharts.min.js:61 Error: <rect> attribute height: Expected length, "NaNNaN".
Currently using chart.yAxis[0].removePlotLine('plotLine') and chart.xAxis[0].addPlotLine(options). I will debug and make a proper report when it remains a problem.

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Oct 9, 2017

Contributor

Hi @atshaas

Thanks! If you could update demo ( http://jsfiddle.net/L20q7aka/1/ - works fine) that would be great! We will look into this.

Contributor

pawelfus commented Oct 9, 2017

Hi @atshaas

Thanks! If you could update demo ( http://jsfiddle.net/L20q7aka/1/ - works fine) that would be great! We will look into this.

@atshaas

This comment has been minimized.

Show comment
Hide comment
@atshaas

atshaas Oct 9, 2017

Our backend bug. We accidentally changed the decimal separator for the threshold value to a comma instead of a full stop. Thus, the threshold and threshold dependent series threw an error on rendering using HighCharts.chart('container', options).

atshaas commented Oct 9, 2017

Our backend bug. We accidentally changed the decimal separator for the threshold value to a comma instead of a full stop. Thus, the threshold and threshold dependent series threw an error on rendering using HighCharts.chart('container', options).

@PrathikGBS

This comment has been minimized.

Show comment
Hide comment
@PrathikGBS

PrathikGBS Jun 1, 2018

is this issue resolved? i am still able to reproduce this.

PrathikGBS commented Jun 1, 2018

is this issue resolved? i am still able to reproduce this.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Jun 1, 2018

Collaborator

It does not reproduce with the original demo: http://jsfiddle.net/highcharts/L20q7aka/15/.

If you can reproduce it on jsFiddle, please share and we'll have a look.

Collaborator

TorsteinHonsi commented Jun 1, 2018

It does not reproduce with the original demo: http://jsfiddle.net/highcharts/L20q7aka/15/.

If you can reproduce it on jsFiddle, please share and we'll have a look.

@PrathikGBS

This comment has been minimized.

Show comment
Hide comment
@PrathikGBS

PrathikGBS Jun 4, 2018

http://jsfiddle.net/L20q7aka/19/
was able to reproduce this in the above fiddle, chk the console error.
am i missing something ?

PrathikGBS commented Jun 4, 2018

http://jsfiddle.net/L20q7aka/19/
was able to reproduce this in the above fiddle, chk the console error.
am i missing something ?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Jun 5, 2018

Contributor

@PrathikGBS

This is caused by yAxis.title.align - available options are 'low', 'high' and 'middle', not 'left': http://jsfiddle.net/BlackLabel/L20q7aka/59/

Contributor

pawelfus commented Jun 5, 2018

@PrathikGBS

This is caused by yAxis.title.align - available options are 'low', 'high' and 'middle', not 'left': http://jsfiddle.net/BlackLabel/L20q7aka/59/

@PrathikGBS

This comment has been minimized.

Show comment
Hide comment
@PrathikGBS

PrathikGBS Jun 5, 2018

tht worked, thanks.

PrathikGBS commented Jun 5, 2018

tht worked, thanks.

@nadim

This comment has been minimized.

Show comment
Hide comment
@nadim

nadim Sep 13, 2018

I'm getting the same error here, but only when I try applying a fullscreen class to the chart:
http://jsfiddle.net/Lxn82jdr/

Any reason why this would happen? The config looks OK to me. Maybe I'm missing something?

Also, not sure why the number 11 is showing on the Y axis. This is something I've been looking into for a while in a number of charts.

Would really appreciate any help, and happy to open another issue report if that's more appropriate.

+Nadim

nadim commented Sep 13, 2018

I'm getting the same error here, but only when I try applying a fullscreen class to the chart:
http://jsfiddle.net/Lxn82jdr/

Any reason why this would happen? The config looks OK to me. Maybe I'm missing something?

Also, not sure why the number 11 is showing on the Y axis. This is something I've been looking into for a while in a number of charts.

Would really appreciate any help, and happy to open another issue report if that's more appropriate.

+Nadim

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Sep 13, 2018

Collaborator

Hi Nadim,

I can't see this error in your demo - what do you mean by applying a fullscreen class? I can't see a reference to that in the code either.

Regarding the number eleven, it is because the Y axis adds another category to make room for the bubbles, and since there is no category 11, it just prints the number. An empty string will fix that: http://jsfiddle.net/highcharts/Lxn82jdr/2/.

Collaborator

TorsteinHonsi commented Sep 13, 2018

Hi Nadim,

I can't see this error in your demo - what do you mean by applying a fullscreen class? I can't see a reference to that in the code either.

Regarding the number eleven, it is because the Y axis adds another category to make room for the bubbles, and since there is no category 11, it just prints the number. An empty string will fix that: http://jsfiddle.net/highcharts/Lxn82jdr/2/.

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