Closed (fixed)
Project:
Conditional Fields
Version:
5.x-1.x-dev
Component:
Javascript
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Jan 2008 at 23:39 UTC
Updated:
24 Apr 2008 at 13:07 UTC
Jump to comment: Most recent file
Comments
Comment #1
bsuttis commentedVersion was wrong =/
Comment #2
Christefano-oldaccount commentedI can confirm that it doesn't work in IE 6.
Comment #3
peterpoe commentedI could use some help with this issue, since I don't have easy access to a Windows machine... Anyone else confirming the bug?
Are the fields not hidden, or not shown? Does this happen with every type of controlling field (checkboxes, radios, select, multiselect)?
Comment #4
Christefano-oldaccount commentedThe fields aren't shown at all when the controlling field is selected. In my case, it was an "Other" checkbox that shows a "Other (please describe)" textfield.
I can do more testing soon.
Comment #5
dragoff commentedTested with IE6 & IE7 (checkbox, radio button, select) - the controlled fields does not show at all. By the way- thanks a ton for the great module :]
Comment #6
scottrigbyI'll be happy to test once a patch is proposed - and I'd be using this module right now on a site except a ton of people who this site appeals to will be using IE :(
Thanks though - looking forward to patches / further releases!
Comment #7
peterpoe commentedThe bug was due to IE not executing the jQuery change event. Maybe this is a jQuery bug? Anyway, I added a switch that ensures that a click event is added with IE (other browsers still use the change event). This could cause other problems, so we have to test the fix thoroughly.
The attached patch is against conditional_fields.js v1.1.2.10.
You can download the patched conditional_fields.js (1.1.2.11) from the latest CVS.
NOTE: the patch was tested ONLY on IE7. Could anyone please test on IE6?
Comment #8
bsuttis commentedFrom my limited testing, it appears to work correctly in IE6/7 and Firefox 2.
Comment #9
peterpoe commentedNo more reported errors in 2 weeks. I'll mark this as fixed.
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #11
patchak commentedHey there, was this added to a new release of the module?? We should use the dev version??
Comment #12
patchak commentedre-opening to have more infos about this....
Comment #13
peterpoe commentedJep, the patch is in the latest dev. Re-closing.