Skip to content
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

Cannot update panes in gauge chart #6089

Closed
PaulDalek opened this issue Dec 5, 2016 · 5 comments
Closed

Cannot update panes in gauge chart #6089

PaulDalek opened this issue Dec 5, 2016 · 5 comments
Labels

Comments

@PaulDalek
Copy link
Contributor

@PaulDalek PaulDalek commented Dec 5, 2016

Behaviour

Function Chart.update() does not seem to update pane in gauge chart. Not sure if this is a bug, as in pre-Chart.update() function era it was not treated as such (#5593)

Live demo

http://jsfiddle.net/d_paul/60dvhrpg/

@oysteinmoseng
Copy link
Member

@oysteinmoseng oysteinmoseng commented Dec 6, 2016

I would definitely consider this a bug, it's not a logical limitation from the user's perspective.

@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented Dec 12, 2016

I did some research and found that chart.update with pane is currently not implemented. I agree it is a bug.

This is how far I got.

The problem now is with the background, which is implemented as a plot band on the pane's first axis. This is a problematic design, especially when it comes to adding, updating and removing, and even on exporting as we have seen before. It is probably not too hard to finish the gist to fix the issue, but we should also consider a different binding between panes and axes.

@newlukai
Copy link

@newlukai newlukai commented Feb 1, 2017

Today I also stumbled upon this bug. How far is the progress here, can we expect a fix soon? Or would I need to create a new Chart object in the case I want to update a gauge chart?

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Feb 6, 2017

+1

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented Mar 31, 2017

Fixed through 2f49339

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants