eclipse-theia/theia

Uncaught Error: Cannot get key code from the keyboard event. Keydown event without code on chrome

Open

#10.702 aberto em 2 de fev. de 2022

Ver no GitHub
 (11 comments) (1 reaction) (0 assignees)TypeScript (2.478 forks)batch import
help wantedkeybindings

Métricas do repositório

Stars
 (18.676 stars)
Métricas de merge de PR
 (Mesclagem média 12d 10h) (73 fundiu PRs em 30d)

Description

Bug Description:

When I click a datalist option linked to an input on Chrome (not Firefox), a keydown event is triggered. This event does not have a key/code/keycode. I don't know if it is a bug of Chrome but in any case, the widget listens keydown events and tries to find the key. Exception is thrown here. The event should probably just be ignored in this case.

This error once happened with electron but it is probably not relevant anymore

Steps to Reproduce:

  1. Create an input and link it with a datalist containing several options in Chrome.
  2. Listen for keydown events.
  3. Click an option
  4. Observe that there is no key information in the event.

Additional Information

  • Operating System: Windows 10, Chrome 97
  • Theia Version: Last, 1.22 (browser)

Guia do colaborador

Direção de pesquisa
O erro ocorre em packages/core/src/common/keys.ts por volta da linha 390. Quando um evento keydown não possui um código de tecla (como acontece na interação com datalist do Chrome), um erro não capturado é lançado. A correção é adicionar uma verificação para ignorar tais eventos. Consulte a issue #8157 para o contexto anterior do Electron. O teste deve envolver a criação de uma entrada com um datalist no Chrome e a verificação de que nenhuma exceção é lançada.
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptkeyboard eventsChrome browser
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75