diff options
| author | Jörn Zaefferer <joern.zaefferer@gmail.com> | 2013-12-03 21:54:08 +0100 |
|---|---|---|
| committer | Jörn Zaefferer <joern.zaefferer@gmail.com> | 2013-12-03 21:54:08 +0100 |
| commit | fe7c3003741cd5221e076235263df2b3d2fa9202 (patch) | |
| tree | c2fa880c62c4454e4f99d8cf37c70362733edd70 /ui | |
| parent | 5494bc9b2e404b2fe6359e7a9f06f2196dbb2940 (diff) | |
| download | jquery-ui-fe7c3003741cd5221e076235263df2b3d2fa9202.tar.gz jquery-ui-fe7c3003741cd5221e076235263df2b3d2fa9202.zip | |
Menu: Remove click binding that was supposed to prevent default on anchor clicks on a disabled menu. Not relevant anymore since we removed the anchors. Partial fix for #8802 - Dev: Track down uses of .bind() inside widgets
Diffstat (limited to 'ui')
| -rw-r--r-- | ui/jquery.ui.menu.js | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/ui/jquery.ui.menu.js b/ui/jquery.ui.menu.js index 86ef4577e..f160f8e6b 100644 --- a/ui/jquery.ui.menu.js +++ b/ui/jquery.ui.menu.js @@ -39,7 +39,8 @@ $.widget( "ui.menu", { _create: function() { this.activeMenu = this.element; - // flag used to prevent firing of the click handler + + // Flag used to prevent firing of the click handler // as the event bubbles up through nested menus this.mouseHandled = false; this.element @@ -49,14 +50,7 @@ $.widget( "ui.menu", { .attr({ role: this.options.role, tabIndex: 0 - }) - // need to catch all clicks on disabled menu - // not possible through _on - .bind( "click" + this.eventNamespace, $.proxy(function( event ) { - if ( this.options.disabled ) { - event.preventDefault(); - } - }, this )); + }); if ( this.options.disabled ) { this.element |