Skip to content

updating time picker should not set vis to dirty - #13793

Merged
ppisljar merged 1 commit into
elastic:masterfrom
ppisljar:fix/categoryAxisTitle
Sep 4, 2017
Merged

ppisljar merged 1 commit into
elastic:masterfrom
ppisljar:fix/categoryAxisTitle

Conversation

@ppisljar

Copy link
Copy Markdown
Contributor

resolves #13770

when i started working on this category axis title was a setting under category axis ... this is some left over code which was causing vis to go dirty every time the timepicker was updated

@ppisljar ppisljar added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v6.0.0 v6.1.0 v7.0.0 labels Aug 31, 2017
@nreese

nreese commented Aug 31, 2017

Copy link
Copy Markdown
Contributor

Since the category axis title is dynamically created each time the visualization is drawn, Why is the title even stored in the visState? Should title.text be removed from visState entirely?

@ppisljar

Copy link
Copy Markdown
Contributor Author

@nreese yes, this PR should remove it

@nreese

nreese commented Aug 31, 2017

Copy link
Copy Markdown
Contributor

It is still in the state.

screen shot 2017-08-31 at 9 53 54 am

@ppisljar

ppisljar commented Sep 1, 2017

Copy link
Copy Markdown
Contributor Author

@nreese is that a new visualization or an old one created before this PR ?

i just tested,

  • created new histogram
  • added date histogram on x axis
  • selected a new time range
  • saved visualization
  • under saved objects i can see title on category axis does not have a text property

@nreese nreese left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ppisljar
ppisljar merged commit 5eb628c into elastic:master Sep 4, 2017
@ppisljar

ppisljar commented Sep 4, 2017

Copy link
Copy Markdown
Contributor Author

backports:
6.x: 2224dad
6.0: 8420159

patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
@github-actions github-actions Bot mentioned this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v6.0.0-rc1 v6.0.0 v6.1.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Save gets disabled if I play with timepicker on linechart and do not click on apply

4 participants