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

Commit

Permalink
fix(tooltip): remove extra digest causing incompatibility
Browse files Browse the repository at this point in the history
In Angular 1.3.1, when using tooltips with ngAnimate, this extra $digest
call seems to break ngClass. This is an extra call to ngAnimate and it
shouldn’t be called since the tooltip hasn’t been added to the DOM.

I was unable to create a test case to test compatibility with ngAnimate.

Fixes #2951
Fixes #2959
  • Loading branch information
chrisirhc committed Nov 14, 2014
1 parent b4832c4 commit 32c4704
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/tooltip/tooltip.js
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,6 @@ angular.module( 'ui.bootstrap.tooltip', [ 'ui.bootstrap.position', 'ui.bootstrap
}
tooltipLinkedScope = ttScope.$new();
tooltip = tooltipLinker(tooltipLinkedScope, angular.noop);
ttScope.$digest();
}

function removeTooltip() {
Expand Down

0 comments on commit 32c4704

Please sign in to comment.