From: timmywil Date: Sun, 13 Mar 2011 20:17:13 +0000 (-0400) Subject: Simplify jQuery.removeAttr and return this X-Git-Tag: 1.6b1~27^2~34 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b85d2cd8a5c49487dc3ea7f6fad963efd9fd0f55;p=jquery.git Simplify jQuery.removeAttr and return this --- diff --git a/src/attributes.js b/src/attributes.js index 64b612617..c19b82923 100644 --- a/src/attributes.js +++ b/src/attributes.js @@ -325,14 +325,15 @@ jQuery.extend({ } }, - // removeAttribute returns boolean in IE - // set property to null if getSetAttribute not supported (IE6-7) removeAttr: function( elem, name ) { name = jQuery.attrFix[ name ] || name; - if ( typeof elem.removeAttribute( name ) === "boolean" && !jQuery.support.getSetAttribute ) { - // Setting className to null sets a class of "null" + + jQuery.support.getSetAttribute ? elem.removeAttribute( name ) : + // set property to null if getSetAttribute not supported (IE6-7) + // setting className to null makes the class "null" name === "className" ? elem.className = "" : elem.setAttribute( name, null ); - } + + return this; }, attrHooks: {