Permalink
Browse files

Selectmenu: unbind Menu document click event (fires unwanted focus ev…

…ent)
  • Loading branch information...
1 parent b356e1e commit c0f9dd31e9ae551f0b4b921435f3ef6981b9da77 @fnagel fnagel committed Feb 24, 2012
Showing with 4 additions and 8 deletions.
  1. +4 −8 ui/jquery.ui.selectmenu.js
View
@@ -154,6 +154,9 @@ $.widget( "ui.selectmenu", {
// change menu styles?
this._setOption( "dropdown", this.options.dropdown );
+ // unbind Menu document event
+ $( document ).unbind( "click.menu" );
+
// document click closes menu
this._bind( document, {
click: function( event ) {
@@ -204,15 +207,8 @@ $.widget( "ui.selectmenu", {
// check if menu has items
if ( this.items ) {
- var currentItem = this._getSelectedItem();
- // needs to be fired after the document click event has closed all other Selectmenus
- // otherwise the current item is not indicated
- // TODO check if this should be handled by Menu
- this._delay( function(){
- this.menu.menu( "focus", event, currentItem );
- }, 1);
-
if ( !this.options.dropdown ) {
+ var currentItem = this._getSelectedItem();
// center current item
if ( this.menu.outerHeight() < this.menu.prop( "scrollHeight" ) ) {
this.menuWrap.css( "left" , -10000 );

0 comments on commit c0f9dd3

Please sign in to comment.