Hiding/showing div clears radio checks

[jimzim]

Hi,

I'm running into a problem where hiding/showing a div clears all checked radios within that div.

I reduced the problem code down to the bare minimum to still exhibit the problem. It's attached, or try it out at ​http://www.thatphonecall.com/jqueryproblem.htm

Is this behavior expected? If so, wouldn't it be more intuitive for show/hide to not clear the checks in radios?

I'm experiencing this problem in IE 6.0.2900.2180...

Thanks, Jim

[jimzim]

Whoops. I left a critical line (line 12) commented out in the attachment. Please disregard the first attachment, and refer only to the 2nd one.

[jimzim]

this is the correct file to refer to

[malsup]

Note that the problem goes away when animation is removed. This works:

$('#hiddenpart').show();

Alerting the 'checked' property of the input shows that it changes from 'true' before the animation to 'undefined' afterwards.

[NatalieMac]

This problem is happening in both IE6 and IE7. It happens when any kind of animation is used to show the container of the radio buttons, but not until the second time the container is shown. Animation to hide the div doesn't affect the checked state of the radio buttons.

[davidserduke]

The problem here appears to be caused when jQuery.css() clones the node and adds it to the form. The code removes the "checked" attribute, but not the "defaultChecked" one. I guess IE decides to check it when it is appeneded if the defaultChecked is set. Adding a removeAttr("defaultChecked") on all radio buttons seems to work at least for this test case.

I'll attach a patch.

[saturnflyer]

I have not applied the attached patch, so take this with a grain of salt...

I did a dirty test of this by applying removeAttr("defaultChecked") to input:radio and reset buttons no longer function properly.

[davidserduke]

Actually if you look closer at the code, the removeAttr("defaultChecked") call is not supposed to be on the actual radio buttons but just on the cloned ones. These cloned ones are just used to get the width/height of the element while it is hidden. So reset will work just fine.

[davidserduke]

Fixed in [3839].

[pkruithof]

This bug is active again in version 1.3.2: performing the test case on a default IE7 setup (not the compatibility view in IE8) leaves both radio buttons unchecked.

[yehuda]

Can't reproduce on head. Added a test for animations.