From ebac9717fd87310fd72c6a17809426a2c111338f Mon Sep 17 00:00:00 2001 From: unknown Date: Wed, 31 Mar 2010 20:17:45 +0200 Subject: [PATCH] Tooltip: Renamed show to open and open to _show (based on spec), changed trigger(show) to trigger(open) and added trigger(close); always use element as target, enabling programmatic opening and closing --- ui/jquery.ui.tooltip.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js index 1fe452450..166bff407 100644 --- a/ui/jquery.ui.tooltip.js +++ b/ui/jquery.ui.tooltip.js @@ -49,7 +49,7 @@ $.widget("ui.tooltip", { this.opacity = this.tooltip.css("opacity"); this.element .bind("focus.tooltip mouseenter.tooltip", function(event) { - self.show($(event.target)); + self.open(); }) .bind("blur.tooltip mouseleave.tooltip", function(event) { self.close(); @@ -73,7 +73,8 @@ $.widget("ui.tooltip", { return this.tooltip; }, - show: function(target) { + open: function() { + var target = this.element; // already visible? possible when both focus and mouseover events occur if (this.current && this.current[0] == target[0]) return; @@ -83,14 +84,14 @@ $.widget("ui.tooltip", { var content = this.options.content.call(target[0], function(response) { // ignore async responses that come in after the tooltip is already hidden if (self.current == target) - self.open(target, response); + self._show(target, response); }); if (content) { - self.open(target, content); + self._show(target, content); } }, - open: function(target, content) { + _show: function(target, content) { if (!content) return; @@ -115,7 +116,7 @@ $.widget("ui.tooltip", { else this.tooltip.is(':visible') ? this.tooltip.fadeTo("normal", this.opacity) : this.tooltip.fadeIn(); - this._trigger("show", null, { target: target }); + this._trigger( "open" ); }, close: function() { @@ -136,6 +137,7 @@ $.widget("ui.tooltip", { else this.tooltip.stop().fadeOut(); + this._trigger( "close" ); } }); -- 2.39.5