Textinput: Rewrote the disable and enable functions to improve readab…

…ility.
jquery-archive · Jul 3, 2012 · aa63dc4 · aa63dc4 · jaspermdegroot · Jul 8, 2012
1 parent 57f9df3
commit aa63dc4
Showing 1 changed file with 16 additions and 6 deletions.
diff --git a/js/widgets/forms/textinput.js b/js/widgets/forms/textinput.js
@@ -138,15 +138,25 @@ $.widget( "mobile.textinput", $.mobile.widget, {
 	},
 
 	disable: function(){
-		( this.element.attr( "disabled", true ).is( "[type='search'],:jqmData(type='search')" ) ?
-			this.element.parent() : this.element ).addClass( "ui-disabled" );
-			return this._setOption( "disabled", true );
+
+		if ( this.element.attr( "disabled", true ).is( "[type='search'],:jqmData(type='search')" ) ) {
+			this.element.parent().addClass( "ui-disabled" );
+		} else {
+			this.element.addClass( "ui-disabled" );
+		}
+		return this._setOption( "disabled", true );
+
 	},
 
 	enable: function(){
-		( this.element.attr( "disabled", false).is( "[type='search'],:jqmData(type='search')" ) ?
-			this.element.parent() : this.element ).removeClass( "ui-disabled" );
-			return this._setOption( "disabled", false );
+
+		if ( this.element.attr( "disabled", false ).is( "[type='search'],:jqmData(type='search')" ) ) {
+			this.element.parent().removeClass( "ui-disabled" );
+		} else {
+			this.element.removeClass( "ui-disabled" );
+		}
+		return this._setOption( "disabled", false );
+
 	}
 });