angular-ui/ui-grid

Memory leak on ui-grid-menu showMenu

Open

#3972 aperta il 13 lug 2015

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)JavaScript (2496 fork)batch import
help wantedseverity: memory leaktype: bug

Metriche repository

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

Descrizione

Hi,

On ui-grid-menu.js, there are 2 methods showMenu and hideMenu that are added to scope but also are added to some self variable that happens to be the ``window` object.

The problem is that in a spa app, navigating away from the list, on the window object that is not going to be destroyed, the 2 functions will remain with the scope as context variable, so all the grid objects, columns, rows will be carried through the rest of the journey.

The 2 functions seem to be used only through scope, so no need for them on the ``window` ... at least to the point I got now.

Guida contributor

Direzione di ricerca
Ispeziona le funzioni showMenu e hideMenu in ui grid menu.js. Conferma che siano attaccate inutilmente all'oggetto window e rimuovi l'assegnazione a window mantenendo l'attaccamento allo scope.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
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
JavaScriptAngularJS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60