deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Non-existent attribute for jQuery.attr no longer needs to check for "undefined"

Browse source 
- Remove an unnecessary var

- Use variable in removeAttr for better minification
This commit is contained in:
timmywil 2011-03-24 00:42:00 -04:00
parent dfeeb872d6
commit 11cfdb2394
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/attributes.js
View file

@ -317,12 +317,12 @@ jQuery.extend({
return ret; return ret;
} else { } else {
var attr = elem.getAttribute( name ); ret = elem.getAttribute( name );
// Non-existent attributes return null, we normalize to undefined // Non-existent attributes return null, we normalize to undefined
return attr === null || attr === "undefined" || attr === "null" ? return ret === null || ret === "null" ?
undefined : undefined :
attr; ret;
} }
} }
}, },
@ -336,7 +336,7 @@ jQuery.extend({
// set property to null if getSetAttribute not supported (IE6-7) // set property to null if getSetAttribute not supported (IE6-7)
// setting className to null makes the class "null" // setting className to null makes the class "null"
if ( name === "className" ) { if ( name === "className" ) {
elem.className = ""; elem[ name ] = "";
} else { } else {
elem.setAttribute( name, null ); elem.setAttribute( name, null );
} }