angular-ui/ui-grid

Scrolling editable rows with down arrow triggers change event with wrong cell values

Open

#6.298 geöffnet am 6. Juli 2017

Auf GitHub ansehen
 (0 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

The example below is originally from the tutorials. Browser is Chrome.

  1. plunkr
  2. select any name
  3. Edit mode is on
  4. now press and hold the down arrow until the grid starts scrolling.
  5. you will now see the details of the last cell changed in the 'last cell edited' field. Although i have not changed anything.

The change should trigger only when the old/new values of the cell has changed. I am simply skipping the 'afterCellEdit' event when the new/old values are the same.

Contributor Guide

Research-Richtung
Untersuche den afterCellEdit Ereignishandler in ui grid, um sicherzustellen, dass Änderungen nur ausgelöst werden, wenn sich Zellenwerte tatsächlich ändern. Der Meldende schlägt vor, das Ereignis zu überspringen, wenn alte und neue Werte identisch sind. Das Problem enthält ein plunkr Beispiel (http://plnkr.co/edit/WizlYTzwdif3Exp1pTDT?p=preview), um den Fehler zu reproduzieren. Betrachte die Zellbearbeitungsfunktionalität des Grids und die Ereignisausbreitung, um die Korrektur zu implementieren.
Tech Stack
javascript
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
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65