From 3ac9eb7ce37b461a34e303e8b77aa544d313441c Mon Sep 17 00:00:00 2001 From: timmywil Date: Thu, 21 Apr 2011 21:33:09 -0400 Subject: Landing pull request [337](https://github.com/jquery/jquery/pull/337). Value of radio inputs resets when type is set after the value in all IEs. Fixes #8570 ([bug](http://bugs.jquery.com/ticket/8570)). --- src/attributes.js | 10 +++++++++- src/support.js | 5 +++++ 2 files changed, 14 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/attributes.js b/src/attributes.js index ac832591a..a296ad112 100644 --- a/src/attributes.js +++ b/src/attributes.js @@ -360,6 +360,15 @@ jQuery.extend({ // We can't allow the type property to be changed (since it causes problems in IE) if ( rtype.test( elem.nodeName ) && elem.parentNode ) { jQuery.error( "type property can't be changed" ); + } 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 + var val = elem.getAttribute("value"); + elem.setAttribute( "type", value ); + if ( val ) { + elem.value = val; + } + return value; } } }, @@ -432,7 +441,6 @@ if ( !jQuery.support.getSetAttribute ) { }); // Use this for any attribute on a form in IE6/7 - // And the name attribute formHook = jQuery.attrHooks.name = jQuery.attrHooks.value = jQuery.valHooks.button = { get: function( elem, name ) { if ( name === "value" && !jQuery.nodeName( elem, "button" ) ) { diff --git a/src/support.js b/src/support.js index 867e18367..b286907ff 100644 --- a/src/support.js +++ b/src/support.js @@ -115,6 +115,11 @@ jQuery.support = (function() { div.cloneNode( true ).fireEvent( "onclick" ); } + input = document.createElement("input"); + input.value = "t"; + input.setAttribute("type", "radio"); + support.radioValue = input.value === "t"; + div.innerHTML = ""; fragment = document.createDocumentFragment(); -- cgit v1.2.3