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

Autocomplete on expression inputs is overzealous #3718

Closed
juliusv opened this Issue Jan 21, 2018 · 9 comments

Comments

Projects
None yet
5 participants
@juliusv
Copy link
Member

juliusv commented Jan 21, 2018

See:

autocomplete

When I type that expression and just press "Return", it will now select a completely unrelated historical expression and execute that instead. It's now quite hard to just query what you want to query when having "Search query history" enabled.

This should be smarter / more restrictive in the matching of history items, and/or it should not auto-select the first completion item, but rather whatever the user has typed (which would require the user to press DownArrow to select the first completion item instead).

@juliusv

This comment has been minimized.

Copy link
Member Author

juliusv commented Jan 21, 2018

/cc @lovisas

@biglovisa

This comment has been minimized.

Copy link
Contributor

biglovisa commented Jan 22, 2018

Not auto selecting the first completion item is something that definitively should be fixed. I'll take a go at making the matching against historical expressions smarter as well.

@juliusv

This comment has been minimized.

Copy link
Member Author

juliusv commented Jan 22, 2018

@lovisas Awesome, thanks!

@biglovisa

This comment has been minimized.

Copy link
Contributor

biglovisa commented Jan 22, 2018

Let me know if this is blocking anyone/anything and I'll split up the work in two smaller PRs to get it through quicker!

@juliusv

This comment has been minimized.

Copy link
Member Author

juliusv commented Jan 22, 2018

@lovisas I guess it'd be good to have something in the next release, but since 2.1.0 just happened, that's probably still quite a ways off.

@beorn7

This comment has been minimized.

Copy link
Member

beorn7 commented Jan 23, 2018

We could consider the current behavior a bug (even a regression as we had the same problem before and then reverted the advanced autocompletion) and then release a fix in 2.1.1.
Or we could just release 2.2 quite soon. As discussed earlier, we are not shy releasing frequently if it delivers some value.

@juliusv

This comment has been minimized.

Copy link
Member Author

juliusv commented Jan 23, 2018

Yeah, I did mark this issue as a bug and consider it one (that really impacts Prometheus use), so doing a 2.1.1 soon would be a good idea.

@paddynewman

This comment has been minimized.

Copy link
Contributor

paddynewman commented Feb 2, 2018

+1 for a 2.1.1 release soon.

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 22, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 22, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.