eclipse-theia/theia

Terminal commands missing in certain instances.

Offen

#8.100 geöffnet am 26.06.2020

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (2.478 Forks)batch import
bugcommandshelp wantedquick-open

Repository-Metriken

Stars
 (18.676 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Bug Description:

The isEnabled handler for many terminal commands may be incorrect. If you give the terminal focus and use CTRL+SHIFT+P you will see commands like clear terminal, however, if you use CTRL+P then type >, you will not see this command. My thinking is that the isEnabled handler is looking at the focused widget, which in the case of CTRL+P, >, is probably the quick open.

Steps to Reproduce:

  1. Use CTRL+P, > with terminal focused, look for Terminal: Clear
  2. Use CTRL+SHIFT+P, with terminal focused, look for Terminal: Clear

Additional Information

  • Operating System: Linux lnx 5.3.0-59-generic #53~18.04.1-Ubuntu SMP Thu Jun 4 14:58:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
  • Theia Version: 1.2

Contributor Guide

Research-Richtung
Verfolgen Sie den isEnabled Handler für Terminalbefehle in terminal frontend contribution.ts und ermitteln Sie, warum er beim Auslösen über CTRL+P, > keine Befehle anzeigt.
Tech Stack
typescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75