diff options
| author | Lukasz Fiszer <[email protected]> | 2013-08-26 20:57:47 +0200 |
|---|---|---|
| committer | Lukasz Fiszer <[email protected]> | 2013-08-26 20:57:47 +0200 |
| commit | 0adb4e35f0d773caa6cff65a6f760001089c5ffb (patch) | |
| tree | cabac5d4ea48156fbdf09b640f84b8d03cd5bcdd | |
| parent | 8f83a6010a05e673c3e0dcfbd97e8b8ddc3d9135 (diff) | |
| download | bootstrap-0adb4e35f0d773caa6cff65a6f760001089c5ffb.tar.xz bootstrap-0adb4e35f0d773caa6cff65a6f760001089c5ffb.zip | |
Fix hidden.bs.tooltip and shown.bs.tooltip events firing too early in tooltip.js
* events fire only after the CSS animation is completed
* this fixes also events in popover.js (as it inherits from tooltip.js)
| -rw-r--r-- | js/tooltip.js | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/js/tooltip.js b/js/tooltip.js index 89802287a..533f19b31 100644 --- a/js/tooltip.js +++ b/js/tooltip.js @@ -139,6 +139,7 @@ this.$element.trigger(e) if (e.isDefaultPrevented()) return + var that = this; var $tip = this.tip() @@ -188,7 +189,17 @@ var calculatedOffset = this.getCalculatedOffset(placement, pos, actualWidth, actualHeight) this.applyPlacement(calculatedOffset, placement) - this.$element.trigger('shown.bs.' + this.type) + + var complete = function() { + that.$element.trigger('shown.bs.' + that.type) + } + + $.support.transition && this.$tip.hasClass('fade') ? + $tip + .one($.support.transition.end, complete) + .emulateTransitionEnd(150) : + complete() + } } @@ -262,6 +273,7 @@ function complete() { if (that.hoverState != 'in') $tip.detach() + that.$element.trigger('hidden.bs.' + that.type) } this.$element.trigger(e) @@ -276,8 +288,6 @@ .emulateTransitionEnd(150) : complete() - this.$element.trigger('hidden.bs.' + this.type) - return this } |
