Skip to content

Comments

Adding secondary-y axis option for line-graph#4585

Closed
gauravsinghania wants to merge 1 commit intoelastic:4.1from
ekstep:multi_y
Closed

Adding secondary-y axis option for line-graph#4585
gauravsinghania wants to merge 1 commit intoelastic:4.1from
ekstep:multi_y

Conversation

@gauravsinghania
Copy link
Contributor

Created an option for the y-axis aggs to be plotted on secondary y-axis for line graphs. Closes #2390 for 4.1

@ghost
Copy link

ghost commented Aug 5, 2015

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

@gauravsinghania gauravsinghania mentioned this pull request Aug 6, 2015
@maheshnayak1
Copy link

Could you please attach a screenshot of working secondary axis?

@gauravsinghania
Copy link
Contributor Author

secondary y axis for first y axis

secondary-y-axis for second y axis

secondary - axis with split chart - columns

secondary - axis with split chart - rows

secondary y axis for 2nd axis - split lines

secondary y axis for 2nd axis with split chart & lines

@gauravsinghania
Copy link
Contributor Author

@rashidkpc any updates?

@gauravsinghania
Copy link
Contributor Author

@spalger any reviews/updates ?

@spalger spalger self-assigned this Aug 20, 2015
@spalger spalger added the review label Aug 20, 2015
@spalger
Copy link
Contributor

spalger commented Aug 20, 2015

@gauravsinghania I'm on vacation for the next couple days, but will be happy to give you some feedback when I return.

@gauravsinghania
Copy link
Contributor Author

@spalger sure, thanks a lot.

@spalger
Copy link
Contributor

spalger commented Aug 25, 2015

@gauravsinghania I didn't realize this was submitted against 4.1. Any chance you can resubmit it against master? We only backport bug fixes to previous versions.

@gauravsinghania
Copy link
Contributor Author

@spalger thanks a lot, didn't know about it. I will try to.

@spalger spalger removed their assignment Sep 1, 2015
@spalger
Copy link
Contributor

spalger commented Sep 1, 2015

When you do, it will need to be as a new pull request so I'll close this for now.

@spalger spalger closed this Sep 1, 2015
@nadest
Copy link

nadest commented Sep 28, 2015

Was this ever implemented?, I am on Kibana 4.2.1 and still not seeing the radio button to add the secondary line for the additional y-axis....?

@dmvk
Copy link

dmvk commented Sep 30, 2015

@gauravsinghania nice! Is there any update on this? It would be really useful feature.

@nadest
Copy link

nadest commented Oct 27, 2015

Any update??

@nadest
Copy link

nadest commented Oct 28, 2015

I can think of so many uses for this, it would so good to have 👍

@bobcat0070
Copy link

I'd love to see this as well. @gauravsinghania Will you have a chance to submit the new pull request?

@gauravsinghania
Copy link
Contributor Author

@bobcat0070 @nadest @davidmoravek I didn't had chance to submit it against the master branch. Will try to submit it some time soon.

@juventietis
Copy link

Two Y axis are a must have for me. Looking forward to this being released.

@jigrons
Copy link

jigrons commented Jan 24, 2016

Is this feature added, if not please add as this is a must for me.

@ruloweb
Copy link

ruloweb commented Apr 19, 2016

This would be a great feature, any new about this? Im using 4.5. Thanks!

@varming
Copy link

varming commented May 12, 2016

@gauravsinghania any chance of you rebasing against master and opening a new pull-request? This would be incredible useful.

FlorianLiers added a commit to FlorianLiers/kibana that referenced this pull request Jun 16, 2016
@FlorianLiers FlorianLiers mentioned this pull request Jun 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants