Skip to content

Commit

Permalink
Combine the support tests for radioValue and appendChecked
Browse files Browse the repository at this point in the history
  • Loading branch information
timmywil committed Apr 22, 2011
1 parent 16312c6 commit 2942be7
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 10 deletions.
1 change: 1 addition & 0 deletions src/attributes.js
Original file line number Diff line number Diff line change
Expand Up @@ -363,6 +363,7 @@ jQuery.extend({
} else if ( !jQuery.support.radioValue && value === "radio" && jQuery.nodeName(elem, "input") ) {
// Setting the type on a radio button after the value resets the value in IE6-9
// Reset value to it's default in case type is set after value
// This is for element creation
var val = elem.getAttribute("value");
elem.setAttribute( "type", value );
if ( val ) {
Expand Down
18 changes: 8 additions & 10 deletions src/support.js
Original file line number Diff line number Diff line change
Expand Up @@ -115,13 +115,15 @@ jQuery.support = (function() {
div.cloneNode( true ).fireEvent( "onclick" );
}

// Check if a radio maintains it's value
// after being appended to the DOM
input = document.createElement("input");
input.value = "t";
input.setAttribute("type", "radio");
support.radioValue = input.value === "t";

div.innerHTML = "<input type='radio' name='radiotest' checked='checked'/>";

input.setAttribute("checked", "checked");
div.appendChild( input );
fragment = document.createDocumentFragment();
fragment.appendChild( div.firstChild );

Expand All @@ -148,6 +150,10 @@ jQuery.support = (function() {
body.appendChild( div );
document.documentElement.appendChild( body );

// Check if a disconnected checkbox will retain its checked
// value of true after appended to the DOM (IE6/7)
support.appendChecked = input.checked;

support.boxModel = div.offsetWidth === 2;

if ( "zoom" in div.style ) {
Expand Down Expand Up @@ -186,14 +192,6 @@ jQuery.support = (function() {
support.reliableHiddenOffsets = isSupported && ( tds[ 0 ].offsetHeight === 0 );
div.innerHTML = "";

// Check if a disconnected checkbox will retain its checked
// value of true after appended to the DOM
input = document.createElement("input");
input.setAttribute("type", "checkbox");
input.checked = true;
div.appendChild( input );
support.appendChecked = input.checked;

// Check if div with explicit width and no margin-right incorrectly
// gets computed margin-right based on width of container. For more
// info see bug #3333
Expand Down

0 comments on commit 2942be7

Please sign in to comment.