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

Move the if statement in jQuery.fn.removeAttr to jQuery.removeAttr

Browse source 
- Extra testing on removeAttr and IE form weirdness( all good )
This commit is contained in:
timmywil 2011-04-03 16:49:48 -04:00
parent ff75767558
commit 2a8a2b6148
1 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/attributes.js
View file

@ -16,9 +16,7 @@ jQuery.fn.extend({
removeAttr: function( name ) { removeAttr: function( name ) {
return this.each(function() { return this.each(function() {
if ( this.nodeType === 1 ) {
jQuery.removeAttr( this, name ); jQuery.removeAttr( this, name );
}
}); });
}, },
@ -335,16 +333,18 @@ jQuery.extend({
}, },
removeAttr: function( elem, name ) { removeAttr: function( elem, name ) {
if ( elem.nodeType === 1 ) {
name = jQuery.attrFix[ name ] || name; name = jQuery.attrFix[ name ] || name;
if ( jQuery.support.getSetAttribute ) { if ( jQuery.support.getSetAttribute ) {
elem.removeAttribute( name ); elem.removeAttribute( name );
} else { } else {
// Set to default empty string // Set to default empty string (No longer need to use attr for this)
elem.setAttribute( name, "" ); elem.setAttribute( name, "" );
// Attempt to remove completely with DOM level 1 // Attempt to remove completely with DOM level 1
elem.removeAttributeNode( elem.getAttributeNode( name ) ); elem.removeAttributeNode( elem.getAttributeNode( name ) );
} }
}
}, },
attrHooks: { attrHooks: {