Skip to content
This repository has been archived by the owner on Jun 14, 2020. It is now read-only.

Commit

Permalink
Removed inconsistency in hoverIntent code that added mouseleave event…
Browse files Browse the repository at this point in the history
… to hide events

Originally the code added a mouseleave event that cleared the show timer
when the show target was first moused over. This was inconsistent with futher
show events where the mouseleave would NOT clear the timer, leading to confusion
over the show.delay behaviour. If you are using a special event like 'unfocus'
that requires a hoverIntent type behaviour, you must added it yourself via some
custom code.
  • Loading branch information
Craga89 committed Aug 17, 2010
1 parent 7270ad7 commit 6f8bd07
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion dist/jquery.qtip.css
Expand Up @@ -9,7 +9,7 @@
* http://en.wikipedia.org/wiki/MIT_License
* http://en.wikipedia.org/wiki/GNU_General_Public_License
*
* Date: Tue Aug 17 14:31:26 2010 +0100
* Date: Tue Aug 17 18:32:10 2010 +0100
*/

.ui-tooltip-accessible{
Expand Down
6 changes: 3 additions & 3 deletions dist/jquery.qtip.js
Expand Up @@ -9,7 +9,7 @@
* http://en.wikipedia.org/wiki/MIT_License
* http://en.wikipedia.org/wiki/GNU_General_Public_License
*
* Date: Tue Aug 17 14:31:26 2010 +0100
* Date: Tue Aug 17 18:32:10 2010 +0100
*/

"use strict"; // Enable ECMAScript "strict" operation for this function. See more: http://ejohn.org/blog/ecmascript-5-strict-mode-json-and-more/
Expand Down Expand Up @@ -425,7 +425,7 @@ function QTip(target, options, id)
var showIndex = $.inArray(type, events.show);

// Both events and targets are identical, apply events using a toggle
if((showIndex > -1 && $(targets.hide).add(targets.show).length === $(targets.hide).length) || type === 'unfocus')
if((showIndex > -1 && $(targets.hide).add(targets.show).length === $(targets.hide).length))
{
targets.show.bind(type+namespace, function(event)
{
Expand Down Expand Up @@ -1201,7 +1201,7 @@ $.fn.qtip.bind = function(opts)
targets = { show: options.show.target, hide: options.hide.target };
events = {
show: String(options.show.event).replace(' ', namespace+' ') + namespace,
hide: (String(options.hide.event) + ' mouseleave').replace(' ', namespace+' ') + namespace
hide: String(options.hide.event).replace(' ', namespace+' ') + namespace
};

// Define hoverIntent function
Expand Down
8 changes: 4 additions & 4 deletions dist/jquery.qtip.min.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 6f8bd07

Please sign in to comment.