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

Add optional tooltip ordering (against the development branch) #4944

Merged
merged 1 commit into from
May 10, 2016

Conversation

jblespiau
Copy link
Contributor

Simply #4867 against the development branch.

After the issue #1189 asking for sorted tooltips, a PR was submitted (#3165) and merged. It was roll-backed because the feature was not optional and just made inspecting series values as you hover impossible as they keep jumping around.

Thus, this PR takes the code of @hcooper and makes it optional. It creates a new select in Misc Options, taking 3 values "Aplhabetical" (default, which is the "name" of current behavior if I am not mistaking), "Increasing" and "Decreasing". The default is the current behavior.
For graphical homogeneity, it also increases the width of the misc options keys to 9 instead of 7.

default
incr
decreasing

@bergquist bergquist self-assigned this May 9, 2016
@bergquist bergquist added the pr/early-feedback WIP state but looking for early feedback label May 9, 2016
@bergquist bergquist merged commit c522341 into grafana:development May 10, 2016
@bergquist
Copy link
Contributor

@Gueust great work! :)

torkelo added a commit that referenced this pull request Jun 15, 2016
@obazoud
Copy link

obazoud commented Jul 20, 2016

is there a way to have the same behaviour for "Legend As Table" ?

@torkelo
Copy link
Member

torkelo commented Jul 20, 2016

If you include legend value you can click on it and sort by it

@obazoud
Copy link

obazoud commented Jul 20, 2016

@torkelo thank for the tip 👍

@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/early-feedback WIP state but looking for early feedback pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants