Vscode: Support custom QuickPick filter logic

Created on 12 Feb 2020  路  5Comments  路  Source: microsoft/vscode

Currently the QuickPick filter logic supports wild cards and which parts of the item to search in but not things like switched up word order or regex. An issue was opened for one of my extensions asking for a more lenient search, which does not seem possible with the current API.

Rather than adding a lot of options specifying exactly how the filtering behaves it might be easier and more sensible to provide a custom filter callback and maybe a filter delay option. The delay would trigger the filter logic only n milliseconds after the user has last typed, in case the filter logic is complex or there are many items (- i have 33k -) causing long filter times.

Suggested interface for the options:

interface QuickPickOptions {
    // ...

    /**
     * Time in milliseconds to wait after last user input
     * before filtering the list.
     * Default: 0
     */
    filterDelay: number,

    /**
     * Optional filter callback which overrides the default filter
     * behavior. The callback is called for every item in the list
     * with the current user input.
     * Default: undefined
     */
    filterCallback?: (item: QuickPickItem | string, filterText: string) => boolean,
}

I had a look at handling a quick pick "manually" by using createQuickPick but it also filters completely automatically.


Maybe QuickPickOptions could also be made generic. It already has the onDidSelectItem property which uses a union for the item type instead of a generic. Then functions like showQuickPick<T extends QuickPickItem> could propagate their generic type to the options argument.

feature-request quick-pick

Most helpful comment

Setting alwaysShow: true on the items and sortByLabel: false (#73904 proposed API) on the QuickPick allows you to control the list and sorting. Only the highlighting cannot be controlled yet, that is tracked with #83424.

Keeping this issue open as its own proposal until the issues are resolved.

All 5 comments

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@brunnerh if we could disable matchOnLabel (see #83425) (as well asmatchOnDescription and matchOnDetail which we already have you could just disable matching and implement the filtering yourself before you put the QuickPickItems into the QuickPick. Coupled with custom highlighting (#83424) you would be able to do whatever you like.

I agree with @brunnerh, about disable match at all, so we could handle the filter ourseves

Setting alwaysShow: true on the items and sortByLabel: false (#73904 proposed API) on the QuickPick allows you to control the list and sorting. Only the highlighting cannot be controlled yet, that is tracked with #83424.

Keeping this issue open as its own proposal until the issues are resolved.

@chrmarti the alwaysShow: true does nos seems to work this way. I have tried to use, but it still keep hiding results based on it owns filter algorithm.

Because of this, i have to do an workaround here:
https://github.com/tatosjb/vscode-fuzzy-search/blob/master/src/search.ts#L66

${this.searchString} - ${path.parse(filePath).dir.replace(workspacePath || '', '')} the ${this.searchString} - on the start of the expression is necessary because of that.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

VitorLuizC picture VitorLuizC  路  3Comments

ryan-wong picture ryan-wong  路  3Comments

lukehoban picture lukehoban  路  3Comments

sijad picture sijad  路  3Comments

biij5698 picture biij5698  路  3Comments