Skip to content

Commit

Permalink
Use prop to retrieve boolean properties (so the selected hook will be…
Browse files Browse the repository at this point in the history
… used)
  • Loading branch information
timmywil committed May 13, 2011
1 parent ec82943 commit 6f676e6
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/attributes.js
Expand Up @@ -466,7 +466,7 @@ jQuery.extend({
boolHook = {
get: function( elem, name ) {
// Align boolean attributes with corresponding properties
return elem[ jQuery.propFix[ name ] || name ] ?
return jQuery.prop( elem, name ) ?
name.toLowerCase() :
undefined;
},
Expand Down
8 changes: 7 additions & 1 deletion test/unit/attributes.js
Expand Up @@ -40,7 +40,7 @@ test("jQuery.attrFix/jQuery.propFix integrity test", function() {
});

test("attr(String)", function() {
expect(37);
expect(38);

equals( jQuery("#text1").attr("type"), "text", "Check for type attribute" );
equals( jQuery("#radio1").attr("type"), "radio", "Check for type attribute" );
Expand Down Expand Up @@ -92,6 +92,12 @@ test("attr(String)", function() {

body.removeAttribute("foo"); // Cleanup

var select = document.createElement("select"), optgroup = document.createElement("optgroup"), option = document.createElement("option");
optgroup.appendChild( option );
select.appendChild( optgroup );

equal( jQuery( option ).attr("selected"), "selected", "Make sure that a single option is selected, even when in an optgroup." );

var $img = jQuery("<img style='display:none' width='215' height='53' src='http://static.jquery.com/files/rocker/images/logo_jquery_215x53.gif'/>").appendTo("body");
equals( $img.attr("width"), "215", "Retrieve width attribute an an element with display:none." );
equals( $img.attr("height"), "53", "Retrieve height attribute an an element with display:none." );
Expand Down

0 comments on commit 6f676e6

Please sign in to comment.