eclipse-theia/theia

Terminal commands missing in certain instances.

Aberta

#8.100 aberto em 26 de jun. de 2020

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)TypeScript (2.478 forks)batch import
bugcommandshelp wantedquick-open

Métricas do repositório

Stars
 (18.676 estrelas)
Métricas de merge de PR
 (Mesclagem média 15d 4h) (75 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Rastreie o manipulador isEnabled para comandos de terminal em terminal frontend contribution.ts e determine por que ele falha ao exibir comandos quando acionado via CTRL+P, >.
Pilha de tecnologia
typescript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75