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

Commit 689c4d0

Browse files
Igosukipkozlowski-opensource
authored andcommitted
fix(tooltip): don't use an empty transclusion fn
Closes #2825
1 parent 6df92c0 commit 689c4d0

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

src/tooltip/tooltip.js

+3-6
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ angular.module( 'ui.bootstrap.tooltip', [ 'ui.bootstrap.position', 'ui.bootstrap
186186
// Set the initial positioning.
187187
tooltip.css({ top: 0, left: 0, display: 'block' });
188188

189-
// Now we add it to the DOM because need some info about it. But it's not
189+
// Now we add it to the DOM because need some info about it. But it's not
190190
// visible yet anyway.
191191
if ( appendToBody ) {
192192
$document.find( 'body' ).append( tooltip );
@@ -214,7 +214,7 @@ angular.module( 'ui.bootstrap.tooltip', [ 'ui.bootstrap.position', 'ui.bootstrap
214214
$timeout.cancel( popupTimeout );
215215
popupTimeout = null;
216216

217-
// And now we remove it from the DOM. However, if we have animation, we
217+
// And now we remove it from the DOM. However, if we have animation, we
218218
// need to wait for it to expire beforehand.
219219
// FIXME: this is a placeholder for a port of the transitions library.
220220
if ( scope.tt_animation ) {
@@ -231,10 +231,7 @@ angular.module( 'ui.bootstrap.tooltip', [ 'ui.bootstrap.position', 'ui.bootstrap
231231
if (tooltip) {
232232
removeTooltip();
233233
}
234-
tooltip = tooltipLinker(scope, function () {});
235-
236-
// Get contents rendered into the tooltip
237-
scope.$digest();
234+
tooltip = tooltipLinker(scope);
238235
}
239236

240237
function removeTooltip() {

0 commit comments

Comments
 (0)