In IE, the show/hide of the "other" input element happens when the checkbox or radio losses focus, instead of when it is clicked. This is due to the different way IE handles the jQuery change event. The attached patch changes "change" to "click", which solves the problem. Issue first reported by kutyamutya.

The patch also removes a call to hide() which is not needed because input elements are either shown or hidden a moment later by select_or_other_check_and_show().

Comments

danielb’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

liam morland’s picture

Version: 6.x-1.2 » 6.x-1.3
Status: Closed (fixed) » Needs review
StatusFileSize
new936 bytes

Unfortunately, my fix for IE broke Chrome/Safari. Attached is a reworked JavaScript file that works for me in Firefox, Safari, Chrome, Opera, and IE 6-7-8. I posted the full file instead of a diff since most of it has changed.

This change means that each div.select-or-other does not require its own @id attribute.

danielb’s picture

Status: Needs review » Fixed

Thanks Liam, I've updated the CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.