angular-ui/ui-grid

Grid is build twice for each data-change when fastWatch is enabled

Open

#6.172 geöffnet am 24. Apr. 2017

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
help wantedseverity: performance

Repository-Metriken

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

Beschreibung

Can be observed on the tutorial page: http://ui-grid.info/docs/#/tutorial/404_large_data_sets_and_performance

Due to these lines (https://github.com/angular-ui/ui-grid/blob/19455276ace7eef5c8dab8f719c17d7c4c31ae3a/src/js/core/directives/ui-grid.js#L56-L57) assigning a new array to the data property rebuilds the rows twice for the same change.

This has a major impact on load times of the grid, when complex rowProcessors are registered (treeBase, sort, etc.)

Contributor Guide

Research-Richtung
Überprüfen Sie den Code in den Zeilen 56-57 von src/js/core/directives/ui grid.js, um die doppelte Zeilenerstellung bei aktiviertem fastWatch zu verstehen.
Tech Stack
angularjavascript
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
JavaScriptAngularJS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70