angular-ui/ui-grid

Incorrect highlighting of first cell when use combination TAB and arrows

Open

#6.399 aberto em 19 de set. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
help wanted

Métricas do repositório

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

Description

When we navigate using both TAB and arrows between cells, focus is shown in the incorrect cell (instead of expected row it's shown in the next row). However, if press ENTER/SPACE, expected check-box will be selected, not that one which was focused. NVDA screen-reader also provides correct information, e.g. we expect to see focus in on row 22, but we see it on the row 23, NVDA says that it's row 22. Possibly, the issue is caused by virtual repeater work, I've debugged this. Focus is set on the correct element, but then it's content changes.

STEPS TO REPRODUCE:

  1. Press TAB and arrows to move focus to the last visible cell.
  2. Press left and right several times but focus must stay in this row and finally appear at last cell.
  3. Press TAB to go to the next row. Expected result: Focus is on first cell of next row. Actual result: Focus is on next + 1 row.

Guia do colaborador

Direção de pesquisa
Investigue a lógica do repetidor virtual no ui grid que re renderiza linhas ao navegar via TAB e teclas de seta. O foco deve seguir o índice de célula pretendido, mas o repetidor virtual pode estar redefinindo o conteúdo da célula após o foco ser definido. Examine os arquivos relacionados à navegação de células e repetidores virtuais, como `src/features/cellnav` e `src/features/virtualRepeater`. Verifique se há uma condição de corrida entre definir o foco e atualizar o DOM. Considere testar com diferentes combinações de TAB e setas para isolar o cenário exato. Nenhum PR ou comentário existente para referência.
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.
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitAngular
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65