diff options
author | Felix Nagel <info@felixnagel.com> | 2013-03-07 19:15:37 +0100 |
---|---|---|
committer | Felix Nagel <info@felixnagel.com> | 2013-03-07 19:15:37 +0100 |
commit | 8787cb045685b93c21c7af6e9ffc6611e679920f (patch) | |
tree | f5c1c5a6901e9cefd50cb4abbd3aaf0807db8447 /ui | |
parent | dffe8f66109714af2d4ed8f582af4cf3439433e3 (diff) | |
download | jquery-ui-8787cb045685b93c21c7af6e9ffc6611e679920f.tar.gz jquery-ui-8787cb045685b93c21c7af6e9ffc6611e679920f.zip |
Selectmenu: switch from a to button element
Diffstat (limited to 'ui')
-rw-r--r-- | ui/jquery.ui.selectmenu.js | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js index 9132604de..568c2e983 100644 --- a/ui/jquery.ui.selectmenu.js +++ b/ui/jquery.ui.selectmenu.js @@ -68,9 +68,8 @@ $.widget( "ui.selectmenu", { this.element.hide(); // create button - this.button = $( "<a>", { + this.button = $( "<button>", { "class": "ui-button ui-widget ui-state-default ui-corner-all", - href: "#" + this.ids.id, tabindex: ( tabindex ? tabindex : this.options.disabled ? -1 : 0 ), id: this.ids.button, width: this.element.outerWidth(), @@ -294,8 +293,12 @@ $.widget( "ui.selectmenu", { this.button.addClass( "ui-state-focus" ); this._off( this.button, "focus" ); }, - click: function( event ) { - this._toggle( event ); + click: function( event ) { + if ( this.isOpen ) { + this.menu.menu( "select", event ); + } else { + this._toggle( event ); + } event.preventDefault(); }, keydown: function( event ) { @@ -327,13 +330,6 @@ $.widget( "ui.selectmenu", { this._move( "next", event ); } break; - case $.ui.keyCode.SPACE: - if ( this.isOpen ) { - this.menu.menu( "select", event ); - } else { - this._toggle( event ); - } - break; case $.ui.keyCode.LEFT: this._move( "previous", event ); break; @@ -348,6 +344,8 @@ $.widget( "ui.selectmenu", { case $.ui.keyCode.PAGE_DOWN: this._move( "last", event ); break; + case $.ui.keyCode.SPACE: + break; default: this.menu.trigger( event ); prevDef = false; |