Skip to content

Commit

Permalink
Mouse: Optimize the cancel locator, works around a bug where .add(eve…
Browse files Browse the repository at this point in the history
…nt.target) in IE8 can take a long time when there are multiple siblings. Fixes #7118 - IE Bug Large ComboBox (Dialog)

(cherry picked from commit 09e88d6)
  • Loading branch information
Jay Merrifield authored and scottgonzalez committed May 27, 2011
1 parent b196d1f commit b04c6ec
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ui/jquery.ui.mouse.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ $.widget("ui.mouse", {

var self = this,
btnIsLeft = (event.which == 1),
elIsCancel = (typeof this.options.cancel == "string" ? $(event.target).parents().add(event.target).filter(this.options.cancel).length : false);
elIsCancel = (typeof this.options.cancel == "string" ? $(event.target).closest(this.options.cancel).length : false);
if (!btnIsLeft || elIsCancel || !this._mouseCapture(event)) {
return true;
}
Expand Down

0 comments on commit b04c6ec

Please sign in to comment.