angular-ui/ui-grid

Memory leak on ui-grid-menu showMenu

已关闭

#3,972 创建于 2015年7月13日

 (1 条评论) (0 个反应) (1 位负责人)JavaScript (2,496 个派生)batch import
help wantedseverity: memory leaktype: bug

仓库指标

星标
 (5,395 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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.

贡献者指南