angular-ui/ui-grid

GridRenderContainer.prototype.getMargin seems to to slowdown apps

Open

#4909 aperta il 27 dic 2015

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)JavaScript (2496 fork)batch import
help wantedseverity: performance

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I profiled one of my apps with Batarang. i noticed that for a page with six smaller ui grid instances the GridRenderContainer.prototype.getMargin takes about 60% of the cpu time.

The method is called from a template:

$templateCache.put('ui-grid/uiGridRenderContainer', ...{ 'margin-left': colContainer.getMargin('left') + 'px', 'margin-right': colContainer.getMargin('right') + 'px' }...

The function loops over viewportAdjusters. Maybe it is worthwhile to optimize the method. For example, it filters out the adjustment results of many of the calls made:

if (adjustment.side && adjustment.side === side)

Such calls could be avoided. Or maybe there is a way to cache the margins and avoid the loop altogether.

Guida contributor

Direzione di ricerca
Esegui il profiling del metodo getMargin per identificare quali viewportAdjusters causano il maggior numero di chiamate. Quindi, considera di memorizzare nella cache i valori dei margini o di filtrare le regolazioni non necessarie prima del ciclo.
Tech stack
javascript
Dominio
frontend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60