angular-ui/ui-grid

Refactor removed ability to handle Enter & arrow keys on grid

Open

#3.805 aberto em 18 de jun. de 2015

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
good first issue

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Refactor: https://github.com/angular-ui/ng-grid/pull/3120 says this:

The problem is that viewPortKeyDown does not get called for navigation keystrokes (specifically Enter and Left/Right Arrow). Previous to this refactor, we could attach an ng-keydown in a cell template to handle these keys.

We desperately need this, as the Enter key lets us submit the form the user is on and the left/right arrow keys let us scroll the grid immediately, irrespective of which column has focus.

I can try to submit a bug fix if that's preferred, though I might not implement it properly...

Guia do colaborador

Direção de pesquisa
Examine o PR #3120 para entender as mudanças de refatoração, em seguida, rastreie o manipulador viewPortKeyDown e a API cellNav para determinar por que os eventos keydown para as teclas Enter/Arrow não estão sendo disparados para navegação. Explore a adição de captura de eventos keydown no template da célula ou estenda viewPortKeyDown para incluir essas teclas.
Pilha de tecnologia
javascriptangular
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
JavaScriptAngularJS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50