From: Alexander Schmitz Date: Fri, 21 Aug 2015 04:06:35 +0000 (-0400) Subject: Selectmenu: Style updates X-Git-Tag: 1.12.0-beta.1~182 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=7e434052696ddf305665816a41c8ec8b4f7c056e;p=jquery-ui.git Selectmenu: Style updates Ref #14246 Ref gh-1588 --- diff --git a/tests/unit/selectmenu/common.js b/tests/unit/selectmenu/common.js index 758e004ba..8582247ca 100644 --- a/tests/unit/selectmenu/common.js +++ b/tests/unit/selectmenu/common.js @@ -21,7 +21,7 @@ common.testWidget( "selectmenu", { }, width: false, - // callbacks + // Callbacks change: null, close: null, create: null, diff --git a/ui/widgets/selectmenu.js b/ui/widgets/selectmenu.js index bf415a826..273e68594 100644 --- a/ui/widgets/selectmenu.js +++ b/ui/widgets/selectmenu.js @@ -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