Skip to content
Permalink
Browse files

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

- Extra testing on removeAttr and IE form weirdness( all good )
  • Loading branch information
timmywil
timmywil committed Apr 3, 2011
1 parent ff75767 commit 2a8a2b61488e7857b116bedc9eb75bd971772c2f
Showing with 11 additions and 11 deletions.
  1. +11 −11 src/attributes.js
@@ -16,9 +16,7 @@ jQuery.fn.extend({

removeAttr: function( name ) {
return this.each(function() {
if ( this.nodeType === 1 ) {
jQuery.removeAttr( this, name );
}
jQuery.removeAttr( this, name );
});
},

@@ -335,15 +333,17 @@ jQuery.extend({
},

removeAttr: function( elem, name ) {
name = jQuery.attrFix[ name ] || name;
if ( elem.nodeType === 1 ) {
name = jQuery.attrFix[ name ] || name;

if ( jQuery.support.getSetAttribute ) {
elem.removeAttribute( name );
} else {
// Set to default empty string
elem.setAttribute( name, "" );
// Attempt to remove completely with DOM level 1
elem.removeAttributeNode( elem.getAttributeNode( name ) );
if ( jQuery.support.getSetAttribute ) {
elem.removeAttribute( name );
} else {
// Set to default empty string (No longer need to use attr for this)
elem.setAttribute( name, "" );
// Attempt to remove completely with DOM level 1
elem.removeAttributeNode( elem.getAttributeNode( name ) );
}
}
},

0 comments on commit 2a8a2b6

Please sign in to comment.
You can’t perform that action at this time.