angular-ui/ui-grid

saveState (in Demo) reselects incorrect rows after sorting (related to: #5663)

Open

#5.706 geöffnet am 29. Sept. 2016

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (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

To reproduce:

  1. go: http://ui-grid.info/docs/#/tutorial/208_save_state
  2. select first three rows
  3. click Save
  4. sort on first column
  5. click Restore

Result: the incorrect three rows are selected.

Observation: as noted in #5663, suspect the use of row indexes is the cause of this issue.

Further: With external pagination, selection or sorting this issue is compounded.

Contributor Guide

Research-Richtung
Untersuche, wie Zeilenindizes während saveState und restore verwendet werden, insbesondere nach dem Sortieren. Siehe das verknüpfte Issue #5663 für Kontext. Überprüfe den Code der saveState Funktion, ob Zeilenreferenzen nach Index statt nach eindeutigen Kennungen gespeichert 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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
AngularGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75