From 3e1557b654e88852ef405422d13eb035dc33e929 Mon Sep 17 00:00:00 2001 From: Felix Nagel Date: Wed, 28 Sep 2011 21:13:14 +0200 Subject: [PATCH] Selectmenu: changed from return false to event.preventDefault() --- ui/jquery.ui.selectmenu.js | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js index a3a81ddcc65..74ef8fa0a02 100644 --- a/ui/jquery.ui.selectmenu.js +++ b/ui/jquery.ui.selectmenu.js @@ -100,18 +100,17 @@ $.widget( "ui.selectmenu", { return false; }, 'keydown.selectmenu': function( event ) { - var ret = true; switch (event.keyCode) { case $.ui.keyCode.TAB: if ( self.opened ) self.close(); break; case $.ui.keyCode.ENTER: if ( self.opened ) self.list.menu( "select", self._getSelectedItem() ); - ret = false; + event.preventDefault(); break; case $.ui.keyCode.SPACE: self._toggle(event); - ret = false; + event.preventDefault(); break; case $.ui.keyCode.UP: if ( event.altKey ) { @@ -119,7 +118,7 @@ $.widget( "ui.selectmenu", { } else { self._move( "previous", event ); } - ret = false; + event.preventDefault(); break; case $.ui.keyCode.DOWN: if ( event.altKey ) { @@ -127,20 +126,19 @@ $.widget( "ui.selectmenu", { } else { self._move( "next", event ); } - ret = false; + event.preventDefault(); break; case $.ui.keyCode.LEFT: self._move( "previous", event ); - ret = false; + event.preventDefault(); break; case $.ui.keyCode.RIGHT: self._move( "next", event ); - ret = false; + event.preventDefault(); break; default: self.list.trigger( event ); } - return ret; } });