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

Opening Spy Panel on Area Chart disables chart Save button #5860

Closed
LeeDr opened this issue Jan 8, 2016 · 5 comments
Closed

Opening Spy Panel on Area Chart disables chart Save button #5860

LeeDr opened this issue Jan 8, 2016 · 5 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience PR sent

Comments

@LeeDr
Copy link
Contributor

LeeDr commented Jan 8, 2016

Found in 4.4.0 on Chrome.

  1. New Visualization
  2. Area Chart
  3. from new or saved search
  4. Set Bucket, X-Axis, Date Histogram, @timestamp, Auto
  5. Apply (if no results change timepicker to get results) - Save button is enabled :-)
  6. click up arrow under chart to open spy panel - Save button is disabled :-(
  7. close spy panel - Save button is still disabled :-(

(same for Pie Chart, I didn't test them all)

@LeeDr LeeDr added the bug Fixes for quality problems that affect the customer experience label Jan 8, 2016
@epixa
Copy link
Contributor

epixa commented Jan 9, 2016

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.

@epixa epixa added the v4.4.0 label Jan 9, 2016
@LeeDr
Copy link
Contributor Author

LeeDr commented Jan 11, 2016

It's the same in 4.3.1.

@epixa epixa removed the v4.4.0 label Jan 11, 2016
@LeeDr LeeDr added the P3 label Jan 11, 2016
@rashidkpc rashidkpc added P2 and removed P3 labels Jan 11, 2016
@stormpython stormpython self-assigned this Jan 11, 2016
@wwsean08
Copy link
Contributor

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)

untitled screencast

@epixa
Copy link
Contributor

epixa commented Jan 12, 2016

Thanks for the details, @wwsean08. That'll help get this thing fixed up quicker.

@stormpython
Copy link
Contributor

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, spyPerPage is added to the editable vis state, but not the actual vis state.

screen shot 2016-01-18 at 6 08 57 pm

And it seems that upon further inspection, the function transferVisState does not get run when the spy is opened. So while the spy adds a parameter to the editable vis state, there is no transfer of state to the actual vis state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience PR sent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants