Feature - Command Palette: Allow specifying arguments for commands · onivim/oni#1960

(1 commento) (0 reazioni) (0 assegnatari)TypeScript (336 fork)batch import

enhancementhelp wantedinsider

Metriche repository

This came up in the context of #1947 - it'd be really convenient when opening a browser window to specify a URL, directly from the command palette:

For example, if I start typing bro and then a space and then http://onivim.io, that would make opening a web browser much more convenient.

Today, we can't do that, since our fuzzy filtering tries to match the whole search string: args-to-command

To implement this, we might need to add a custom filter strategy for the command palette window (such that it only uses the first token in the string), and then update the Tasks provider (which backs the command palette) to send back any arguments. That code is around here: https://github.com/onivim/oni/blob/8186d0ba5754282b81464748355ad702d5a4f8e5/browser/src/Services/Tasks.ts#L78

Direzione di ricerca: Studia l'implementazione attuale della palette dei comandi, in particolare il filtro fuzzy nel provider Tasks in browser/src/Services/Tasks.ts. Comprendi come i comandi vengono abbinati ed eseguiti. Considera l'aggiunta di una strategia di filtro personalizzata che utilizza solo il primo token per la corrispondenza e passa i token rimanenti come argomenti al comando. Inoltre, rivedi eventuali test esistenti per la palette dei comandi per garantire che non ci siano regressioni.
Tech stack: typescript
Dominio: frontend
Tipo issue: Funzionalità
Difficoltà: 3
Tempo stimato: Mezza giornata
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: TypeScript
Adatta ai principianti: 60