angular-ui/ui-grid

Incorrect highlighting of first cell when use combination TAB and arrows

Open

#6.399 geöffnet am 19. Sept. 2017

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
help wanted

Repository-Metriken

Stars
 (5.395 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Logik des virtuellen Repeaters in ui grid, die Zeilen bei der Navigation über TAB und Pfeiltasten neu rendert. Der Fokus sollte dem beabsichtigten Zellenindex folgen, aber der virtuelle Repeater könnte den Zelleninhalt zurücksetzen, nachdem der Fokus gesetzt wurde. Sieh dir Dateien im Zusammenhang mit Zellennavigation und virtuellen Repeatern an, wie `src/features/cellnav` und `src/features/virtualRepeater`. Prüfe, ob es eine Wettlaufsituation zwischen dem Setzen des Fokus und dem Aktualisieren des DOM gibt. Erwäge, mit verschiedenen Kombinationen von TAB und Pfeiltasten zu testen, um das genaue Szenario zu isolieren. Keine bestehenden PRs oder Kommentare als Referenz.
Tech Stack
javascriptangular
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitAngular
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65