Skip to content

Conversation

@ppisljar
Copy link
Contributor

fixes #9114 where when you set a secondary y axis it would remove all the configuration you set on the first one (so the first one behaves like a default axis)

for example
.static(6000).yaxis(min=5000, position=right), .static(3000).yaxis(2, position=left)

will show both axes on the left (position=right on the first axis will get ignored, as well as the minimum)

@ppisljar ppisljar added Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) labels Nov 23, 2016
@rashidkpc
Copy link
Contributor

Clearly LGTM. Merge it.

@rashidkpc rashidkpc assigned ppisljar and unassigned rashidkpc Nov 23, 2016
@ppisljar ppisljar merged commit 11f5711 into elastic:master Nov 23, 2016
@epixa epixa added v5.1.0 and removed v5.2.0 labels Nov 23, 2016
elastic-jasper added a commit that referenced this pull request Nov 23, 2016
Backports PR #9197

**Commit 1:**
fixing timelion Y axis #9114

* Original sha: 54107d6
* Authored by ppisljar <[email protected]> on 2016-11-23T09:37:52Z
@epixa epixa changed the title [timelion] fixing Y axis #9114 [timelion] fixing Y axis Nov 23, 2016
@epixa epixa changed the title [timelion] fixing Y axis [timelion] Secondary y-axis no longer removes config on first axis Nov 23, 2016
epixa pushed a commit that referenced this pull request Nov 23, 2016
Backports PR #9197

**Commit 1:**
fixing timelion Y axis #9114

* Original sha: 54107d6
* Authored by ppisljar <[email protected]> on 2016-11-23T09:37:52Z
elastic-jasper added a commit that referenced this pull request Nov 23, 2016
Backports PR #9197

**Commit 1:**
fixing timelion Y axis #9114

* Original sha: 54107d6
* Authored by ppisljar <[email protected]> on 2016-11-23T09:37:52Z
@epixa epixa added the v5.2.0 label Nov 23, 2016
epixa pushed a commit that referenced this pull request Nov 23, 2016
…9200)

Backports PR #9197

**Commit 1:**
fixing timelion Y axis #9114

* Original sha: 54107d6
* Authored by ppisljar <[email protected]> on 2016-11-23T09:37:52Z
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
…lastic#9200)

Backports PR elastic#9197

**Commit 1:**
fixing timelion Y axis elastic#9114

* Original sha: 54107d6
* Authored by ppisljar <[email protected]> on 2016-11-23T09:37:52Z

Former-commit-id: accc45c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.1.1 v5.2.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

timelion secondary y axis removes configuration of the first one

4 participants