eclipse-theia/theia

Terminal commands missing in certain instances.

Ouverte

#8 100 ouverte le 26 juin 2020

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)TypeScript (2 478 forks)batch import
bugcommandshelp wantedquick-open

Métriques du dépôt

Stars
 (18 676 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Tracez le gestionnaire isEnabled pour les commandes de terminal dans terminal frontend contribution.ts et déterminez pourquoi il échoue à afficher les commandes lorsqu'il est déclenché via CTRL+P, >.
Stack technique
typescript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75