onivim/oni

Feature - Command Palette: Allow specifying arguments for commands

Open

#1.960 geöffnet am 28. März 2018

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (336 Forks)batch import
enhancementhelp wantedinsider

Repository-Metriken

Stars
 (11.389 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Erforsche die aktuelle Implementierung der Befehlspalette, insbesondere den Fuzzy Filter im Tasks Provider unter browser/src/Services/Tasks.ts. Verstehe, wie Befehle abgeglichen und ausgeführt werden. Erwäge, eine benutzerdefinierte Filterstrategie hinzuzufügen, die nur das erste Token für den Abgleich verwendet und die verbleibenden Token als Argumente an den Befehl weitergibt. Überprüfe außerdem vorhandene Tests für die Befehlspalette, um sicherzustellen, dass keine Regressionen auftreten.
Tech Stack
typescript
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60