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

(1 commentaire) (0 réactions) (0 assignés)TypeScript (336 forks)batch import

enhancementhelp wantedinsider

Métriques du dépôt

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

Direction de recherche: Recherchez l'implémentation actuelle de la palette de commandes, en particulier le filtre flou dans le fournisseur de Tâches à browser/src/Services/Tasks.ts. Comprenez comment les commandes sont appariées et exécutées. Envisagez d'ajouter une stratégie de filtre personnalisée qui n'utilise que le premier jeton pour l'appariement et passe les jetons restants comme arguments à la commande. De plus, examinez tous les tests existants pour la palette de commandes pour garantir qu'il n'y a pas de régressions.
Stack technique: typescript
Domaine: frontend
Type d'issue: Fonctionnalité
Difficulté: 3
Temps estimé: Une demi journée
Statut d'activité: Active
Clarté: Claire
Prérequis: TypeScript
Accessibilité débutant: 60