fixed: its now possible to have a disabled and selected option on ini…

…t, see jquery#234
blablacio · Jul 20, 2012 · 690fcb3 · 690fcb3
1 parent c67774f
commit 690fcb3
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/demos/selectmenu/disable_enable.html b/demos/selectmenu/disable_enable.html
@@ -67,7 +67,9 @@
 
 			var speedC = $('select#speedC').selectmenu();
 
-			var speedD = $('select#speedD').selectmenu();		
+			var speedD = $('select#speedD').selectmenu();
+
+			var speedD_first = $('select#speedD_first').selectmenu();		
 
 			var speedD2 = $('select#speedD2').selectmenu();				
 			$("#optgroupSwitch").toggle(
@@ -169,6 +171,16 @@ <h3>HTML</h3>
 			</select>
 		</fieldset>
 
+		<fieldset>
+			<label for="speedD_first">Disabled and selected first option by HTML</label>
+			<select id="speedD_first">
+				<option disabled="disabled" selected="selected">Choose one...</option>
+				<option>Option 1</option>
+				<option>Option 2</option>
+				<option>Option 3</option>
+			<select>
+		</fieldset>
+
 		<fieldset>
 			<label for="speedD3">Disabled optgroup by HTML</label>
 			<select name="speedD3" id="speedD3">

diff --git a/ui/jquery.ui.selectmenu.js b/ui/jquery.ui.selectmenu.js
@@ -436,7 +436,7 @@ $.widget("ui.selectmenu", {
 		}
 
 		// update value
-		this.index( this._selectedIndex() );
+		this._refreshValue();
 
 		// set selected item so movefocus has intial state
 		this._selectedOptionLi().addClass(this.widgetBaseClass + '-item-focus');