angular-ui/ui-grid

Async row removal does not work with inline edit

Open

#5.649 geöffnet am 26. Aug. 2016

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

We remove large amounts of data by a background worker and we reflect this on ui-grid. So rows might be removed without any user action (just the refresher which runs in background notices that row is no longer present in database, it is also removed from grid immediately).

The problem is that if user is inline editing any row BELOW such removed row, inline edit still keeps old index.

I have created a plunkr to demonstrate the issue: http://plnkr.co/edit/I8KnXLCxeDiKQq5OR9lC?p=preview

Steps to reproduce:

  1. Start inline edit of second row immediately (e.g. change name and remain in inline edit mode)
  2. After 10 seconds first row is automatically removed
  3. Observe that row with the same index is in edit mode, but actually edited row is moved up. So user is editing another row.

Contributor Guide

Research-Richtung
Reproduzieren Sie das Problem mit dem bereitgestellten Plunkr und verfolgen Sie dann die Behandlung des Inline Bearbeitungsindex, wenn Zeilen asynchron entfernt werden.
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.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptAngular
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75