Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

fix(tooltip): memory leak on show/hide #2919

Closed
wants to merge 2 commits into from
Closed

fix(tooltip): memory leak on show/hide #2919

wants to merge 2 commits into from

Commits on Nov 2, 2014

  1. fix(tooltip): remove child scope requirement 

    tt_ scope variables are now in the ttScope which is specific to each
tooltip directive (multiple tooltip directives can now run on the same
element).

This allows tooltips to be used alongside any directive that may or
may not require different scope types.

Closes angular-ui#1269
Fixes angular-ui#2320
Fixes angular-ui#2203
    @chrisirhc
    chrisirhc committed Nov 2, 2014
    Configuration menu
    Copy the full SHA
    ebe4119 View commit details
    Browse the repository at this point in the history

  2. fix(tooltip): memory leak on show/hide 

    Create a new child scope and retain a reference to it so that it can
be destroyed when the tooltip DOM is removed.

Fixes angular-ui#2709
    @chrisirhc
    chrisirhc committed Nov 2, 2014
    Configuration menu
    Copy the full SHA
    5629bfa View commit details
    Browse the repository at this point in the history