From: Jörn Zaefferer <joern.zaefferer@gmail.com>
Date: Mon, 21 Sep 2015 11:01:13 +0000 (+0200)
Subject: Selectmenu: Remove redundant handling of disabled option on create
X-Git-Tag: 1.12.0-beta.1~111
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d2f62041f97f5cc31f191c4f5e155e336a035878;p=jquery-ui.git

Selectmenu: Remove redundant handling of disabled option on create

Ref #9151
Ref gh-1599
---

diff --git a/ui/widgets/selectmenu.js b/ui/widgets/selectmenu.js
index 820e68570..30d8c71af 100644
--- a/ui/widgets/selectmenu.js
+++ b/ui/widgets/selectmenu.js
@@ -79,10 +79,6 @@ return $.widget( "ui.selectmenu", {
 
 		this._rendered = false;
 		this.menuItems = $();
-
-		if ( this.options.disabled ) {
-			this.disable();
-		}
 	},
 
 	_drawButton: function() {
@@ -562,25 +558,27 @@ return $.widget( "ui.selectmenu", {
 			this.menuWrap.appendTo( this._appendTo() );
 		}
 
-		if ( key === "disabled" ) {
-			this.menuInstance.option( "disabled", value );
-			this.button.attr( "aria-disabled", value );
-			this._toggleClass( this.button, null, "ui-state-disabled", value );
-
-			this.element.prop( "disabled", value );
-			if ( value ) {
-				this.button.attr( "tabindex", -1 );
-				this.close();
-			} else {
-				this.button.attr( "tabindex", 0 );
-			}
-		}
-
 		if ( key === "width" ) {
 			this._resizeButton();
 		}
 	},
 
+	_setOptionDisabled: function( value ) {
+		this._super( value );
+
+		this.menuInstance.option( "disabled", value );
+		this.button.attr( "aria-disabled", value );
+		this._toggleClass( this.button, null, "ui-state-disabled", value );
+
+		this.element.prop( "disabled", value );
+		if ( value ) {
+			this.button.attr( "tabindex", -1 );
+			this.close();
+		} else {
+			this.button.attr( "tabindex", 0 );
+		}
+	},
+
 	_appendTo: function() {
 		var element = this.options.appendTo;