library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Menu: Handle clicking on a menu item without hovering it first. Fixes #7085 - Menu click when no mouseover causes error.

Browse Source
This commit is contained in:
Scott González 2011-03-15 14:02:18 -04:00
parent 9b3badea6c
commit da67ece506
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ui/jquery.ui.menu.js vendored
View File

@ -34,14 +34,19 @@ $.widget("ui.menu", {
role: "listbox" role: "listbox"
}) })
.bind( "click.menu", function( event ) { .bind( "click.menu", function( event ) {
var item = $( event.target ).closest( ".ui-menu-item:has(a)" );
if ( self.options.disabled ) { if ( self.options.disabled ) {
return false; return false;
} }
if ( !$( event.target ).closest( ".ui-menu-item a" ).length ) { if ( !item.length ) {
return; return;
} }
// temporary // temporary
event.preventDefault(); event.preventDefault();
// it's possible to click an item without hovering it (#7085)
if ( !self.active || ( self.active[ 0 ] !== item[ 0 ] ) ) {
self.focus( event, item );
}
self.select( event ); self.select( event );
}) })
.bind( "mouseover.menu", function( event ) { .bind( "mouseover.menu", function( event ) {
@ -102,7 +107,7 @@ $.widget("ui.menu", {
event.preventDefault(); event.preventDefault();
break; break;
case $.ui.keyCode.ENTER: case $.ui.keyCode.ENTER:
self.select(); self.select( event );
event.preventDefault(); event.preventDefault();
event.stopImmediatePropagation(); event.stopImmediatePropagation();
break; break;