Skip to content
Permalink
Browse files

Selectmenu: Style updates

Ref #14246
Ref gh-1588
  • Loading branch information
arschmitz committed Aug 21, 2015
1 parent 1d572f0 commit 7e434052696ddf305665816a41c8ec8b4f7c056e
Showing with 7 additions and 7 deletions.
  1. +1 −1 tests/unit/selectmenu/common.js
  2. +6 −6 ui/widgets/selectmenu.js
@@ -21,7 +21,7 @@ common.testWidget( "selectmenu", {
},
width: false,

// callbacks
// Callbacks
change: null,
close: null,
create: null,
@@ -58,7 +58,7 @@ return $.widget( "ui.selectmenu", {
},
width: false,

// callbacks
// Callbacks
change: null,
close: null,
focus: null,
@@ -169,7 +169,7 @@ return $.widget( "ui.selectmenu", {
select: function( event, ui ) {
event.preventDefault();

// support: IE8
// Support: IE8
// If the item was selected via a click, the text selection
// will be destroyed in IE
that._setSelection();
@@ -410,12 +410,12 @@ return $.widget( "ui.selectmenu", {
selection.removeAllRanges();
selection.addRange( this.range );

// support: IE8
// Support: IE8
} else {
this.range.select();
}

// support: IE
// Support: IE
// Setting the text selection kills the button focus in IE, but
// restoring the focus doesn't kill the selection.
this.button.focus();
@@ -446,7 +446,7 @@ return $.widget( "ui.selectmenu", {
this.range = selection.getRangeAt( 0 );
}

// support: IE8
// Support: IE8
} else {
this.range = document.selection.createRange();
}
@@ -638,7 +638,7 @@ return $.widget( "ui.selectmenu", {
this.menu.outerWidth( Math.max(
this.button.outerWidth(),

// support: IE10
// Support: IE10
// IE10 wraps long text (possibly a rounding bug)
// so we add 1px to avoid the wrapping
this.menu.width( "" ).outerWidth() + 1

0 comments on commit 7e43405

Please sign in to comment.
You can’t perform that action at this time.