Permalink
Browse files

Selectmenu: Switch to span to avoid cranky click/keydown event handling

  • Loading branch information...
1 parent 6e80a19 commit 022079a7821e5c34583b5a34710fbaeaae81cd15 @jzaefferer jzaefferer committed Mar 7, 2013
Showing with 3 additions and 2 deletions.
  1. +3 −2 ui/jquery.ui.selectmenu.js
@@ -68,7 +68,7 @@ $.widget( "ui.selectmenu", {
this.element.hide();
// create button
- this.button = $( "<button>", {
+ this.button = $( "<span>", {
"class": "ui-button ui-widget ui-state-default ui-corner-all",
tabindex: ( tabindex ? tabindex : this.options.disabled ? -1 : 0 ),
id: this.ids.button,
@@ -332,8 +332,9 @@ $.widget( "ui.selectmenu", {
case $.ui.keyCode.SPACE:
if ( this.isOpen ) {
this.menu.menu( "select", event );
+ } else {
+ this._toggle( event );
}
- prevDef = false;
break;
case $.ui.keyCode.LEFT:
this._move( "previous", event );

0 comments on commit 022079a

Please sign in to comment.