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

Improve the sorting and matching of quick pick #14879

Closed
kentcdodds opened this issue Nov 2, 2016 · 9 comments
Closed

Improve the sorting and matching of quick pick #14879

kentcdodds opened this issue Nov 2, 2016 · 9 comments
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality quick-pick Quick-pick widget issues

Comments

@kentcdodds
Copy link

Originally posted in the vscode-project-manager extension issues here. @alefragnani suggested that I file the issue here to have this applied to PickList. I'm unfamiliar with this codebase. But I think that match-sorter provides a better way to filter and sort items in a list based on user input. Could we consider adding that kind of functionality to VSCode?

Thanks!

@jrieken jrieken assigned bpasero and unassigned jrieken Nov 3, 2016
@bpasero
Copy link
Member

bpasero commented Nov 3, 2016

@kentcdodds is this about general sorting in all quick open things like file search, command search etc. or only for extensions that use the picker widget?

@bpasero bpasero added info-needed Issue requires more information from poster feature-request Request for new features or functionality labels Nov 3, 2016
@bpasero bpasero added this to the Backlog milestone Nov 3, 2016
@kentcdodds
Copy link
Author

I think that the way match-sorter works too improve the intuitiveness of lost filtering and matching is actually pretty great, so it probably makes sense for everything.

@bpasero bpasero changed the title Improve sorting/filtering of PickList Improve the sorting and matching of quick open Nov 3, 2016
@bpasero bpasero added workbench and removed info-needed Issue requires more information from poster labels Nov 3, 2016
@bpasero bpasero removed this from the Backlog milestone Nov 3, 2016
@bpasero bpasero removed their assignment Nov 3, 2016
@bpasero
Copy link
Member

bpasero commented Nov 3, 2016

Makes sense.

@alefragnani
Copy link

alefragnani commented Nov 3, 2016

@bpasero, When I asked @kentcdodds to fill an issue here, I didn't remember the other issue that already existed, so it could be linked. I did another search and found, it's #14727.

Since you were looking for a better fuzzy search, I thought you could work together on this.

Hope this helps

@bpasero
Copy link
Member

bpasero commented Nov 3, 2016

Yup, they are related. I am however currently not working on it (I unassigned myself). Sorry for the confusion.

@atombender
Copy link

Here's an example of where it goes wrong:

e8b7bd8a-aa6b-11e6-8188-78c294ce5efd

The file cmd/gradient/main.go is scored lower even though the text I typed matches contiguously. The top matches have matched random characters in other file names which seem to have accidentally have bumped their score up, even though the matches make less sense.

The solution is to score in a way that promotes results with a low distance between each character. Levenshtein would work here.

@kentcdodds
Copy link
Author

Yeah, there's an issue right now for adding something like that to match-sorter here. FWIW, given that screenshot, match-sorter would have sorted things more cleanly.

@bpasero bpasero changed the title Improve the sorting and matching of quick open Improve the sorting and matching of quick pick Sep 29, 2017
@bpasero bpasero added the quick-pick Quick-pick widget issues label Sep 29, 2017
@bpasero bpasero removed the workbench label Nov 14, 2017
@bpasero
Copy link
Member

bpasero commented Nov 17, 2017

/duplicate #34088

@vscodebot vscodebot bot added the *duplicate Issue identified as a duplicate of another issue(s) label Nov 17, 2017
@vscodebot
Copy link

vscodebot bot commented Nov 17, 2017

This issue has been closed because it is already tracked by another issue. See also our GitHub issues to search for existing issues and our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Nov 17, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality quick-pick Quick-pick widget issues
Projects
None yet
Development

No branches or pull requests

5 participants