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

Highcharts: solid gauge chart yAxis update #6433

Closed
xorsnn opened this Issue Mar 6, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@xorsnn

xorsnn commented Mar 6, 2017

Expected behaviour

When updating yAxis using

chartSpeed.yAxis[0].setTitle({ text: "The new titile" });

I expect dom consistency, and expect that additional unsued nodes won't appear

Actual behaviour

every time yAxis is updated new dom node appears:
<g class="highcharts-plot-bands-0"></g>
output

Live demo with steps to reproduce

http://jsfiddle.net/xors/afq7ge9v/8/

Affected browser(s)

google chrom 56.0.2924.87 (64-bit)

@xorsnn xorsnn changed the title from Highcharts: solig gauge chart yAxis update to Highcharts: solid gauge chart yAxis update Mar 6, 2017

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 13, 2017

Contributor

Hi @xorsnn

Thank your for reporting this issue.

Internal note:
It can be caused by pane's background, when disabled it works fine: http://jsfiddle.net/afq7ge9v/9/

Edit: It's a general issue, not only applied to gauge, demo: http://jsfiddle.net/bnob0qhh/

Contributor

pawelfus commented Mar 13, 2017

Hi @xorsnn

Thank your for reporting this issue.

Internal note:
It can be caused by pane's background, when disabled it works fine: http://jsfiddle.net/afq7ge9v/9/

Edit: It's a general issue, not only applied to gauge, demo: http://jsfiddle.net/bnob0qhh/

@pawelfus pawelfus added the Bug label Mar 13, 2017

@pawelfus pawelfus self-assigned this Mar 16, 2017

pawelfus added a commit that referenced this issue Mar 16, 2017

pawelfus added a commit that referenced this issue Mar 16, 2017

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 16, 2017

Contributor

Internal note:
There are two commits to add test, but only the second one is correct. Reverting went wrong..

Contributor

pawelfus commented Mar 16, 2017

Internal note:
There are two commits to add test, but only the second one is correct. Reverting went wrong..

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