-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Opening Spy Panel on Area Chart disables chart Save button #5860
Comments
We need to confirm whether this is new to 4.4.0 or not. For now, I'm going to put the 4.4.0 label on it so we don't forget. |
It's the same in 4.3.1. |
So with some testing I found that this is caused because when you open the spy panel it marks the vis's state as dirty, whenever that happens the save button is disabled. You can see that when you do this the apply button becomes active again. If you click the apply button it "cleans" the state and you are able to save. That should at least give a workaround for now. I also noticed that after hitting the apply button subsequent openings and closings of the spy panel did not affect the ability to save the visualization. Here's a recording of what I mean. (for some reason the software didn't show the normal cursor so you can't see the no symbol or whatever you wana call it) |
Thanks for the details, @wwsean08. That'll help get this thing fixed up quicker. |
Thanks @wwsean08 for the head start. So the issue here seems to occur at lines 128-133. The state is set to dirty because a vis spy parameter, And it seems that upon further inspection, the function |
…f on the editableVis.params.
…e editableVis.params. Fixes #5942
…e editableVis.params. Fixes #5942
Found in 4.4.0 on Chrome.
(same for Pie Chart, I didn't test them all)
The text was updated successfully, but these errors were encountered: