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

Set corresponding property to false when removing boolean attributes. Fixes #9094

1.7/enhancement_8685
timmywil 2011-05-04 11:29:38 -04:00
parent 97144424cc
commit 480b88ca37
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/attributes.js
View File

@ -369,6 +369,12 @@ jQuery.extend({
jQuery.attr( elem, name, "" );
elem.removeAttributeNode( elem.getAttributeNode( name ) );
}
// Set corresponding property to false for boolean attributes
name = jQuery.propFix[ name ] || name;
if ( !rinvalidChar.test( name ) && typeof elem[ name ] === "boolean" ) {
elem[ name ] = false;
}
}
},

7
test/unit/attributes.js
View File

@ -405,12 +405,17 @@ test("attr('tabindex', value)", function() {
});
test("removeAttr(String)", function() {
expect(5);
expect(7);
equals( jQuery("#mark").removeAttr( "class" )[0].className, "", "remove class" );
equals( jQuery("#form").removeAttr("id").attr("id"), undefined, "Remove id" );
equals( jQuery("#foo").attr("style", "position:absolute;").removeAttr("style").attr("style"), undefined, "Check removing style attribute" );
equals( jQuery("#form").attr("style", "position:absolute;").removeAttr("style").attr("style"), undefined, "Check removing style attribute on a form" );
equals( jQuery("#fx-test-group").attr("height", "3px").removeAttr("height").css("height"), "1px", "Removing height attribute has no effect on height set with style attribute" );
jQuery("#check1").removeAttr("checked").prop("checked", true).removeAttr("checked");
equals( document.getElementById("check1").checked, false, "removeAttr sets boolean properties to false" );
jQuery("#text1").prop("readOnly", true).removeAttr("readonly");
equals( document.getElementById("text1").readOnly, false, "removeAttr sets boolean properties to false" );
});
test("prop(String, Object)", function() {