From 11cfdb23940b2c76e1104c82afc9552ac6f3dc19 Mon Sep 17 00:00:00 2001 From: timmywil Date: Thu, 24 Mar 2011 00:42:00 -0400 Subject: [PATCH] Non-existent attribute for jQuery.attr no longer needs to check for "undefined" - Remove an unnecessary var - Use variable in removeAttr for better minification --- src/attributes.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/attributes.js b/src/attributes.js index 55d22eb9..dc9a333b 100644 --- a/src/attributes.js +++ b/src/attributes.js @@ -317,12 +317,12 @@ jQuery.extend({ return ret; } else { - var attr = elem.getAttribute( name ); + ret = elem.getAttribute( name ); // Non-existent attributes return null, we normalize to undefined - return attr === null || attr === "undefined" || attr === "null" ? + return ret === null || ret === "null" ? undefined : - attr; + ret; } } }, @@ -336,7 +336,7 @@ jQuery.extend({ // set property to null if getSetAttribute not supported (IE6-7) // setting className to null makes the class "null" if ( name === "className" ) { - elem.className = ""; + elem[ name ] = ""; } else { elem.setAttribute( name, null ); }