angular-ui/ui-grid

Refactor removed ability to handle Enter & arrow keys on grid

Open

#3.805 geöffnet am 18. Juni 2015

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
good first issue

Repository-Metriken

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

Beschreibung

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...

Contributor Guide

Research-Richtung
Untersuche PR #3120, um die Refactoring Änderungen zu verstehen, und verfolge dann den viewPortKeyDown Handler und die cellNav API, um festzustellen, warum die keydown Ereignisse für die Eingabe /Pfeiltasten nicht für die Navigation ausgelöst werden. Erkunde das Hinzufügen von keydown Ereigniserfassung in der Zellvorlage oder die Erweiterung von viewPortKeyDown, um diese Tasten einzuschließen.
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.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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
JavaScriptAngularJS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50