Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I've got this working just fine in FF 2, but on both a fresh install of 5.5 and a production site running 5.1 in IE6/7, selecting the conditional value that should display the hidden field doesn't work.
I'm guessing this is just a javascript discrepancy? Anyone have fields showing/hiding in IE6/7?
Comment | File | Size | Author |
---|---|---|---|
#7 | conditional_fields.js IE patch | 3.62 KB | peterpoe |
Comments
Comment #1
bsuttis CreditAttribution: bsuttis commentedVersion was wrong =/
Comment #2
Christefano-oldaccount CreditAttribution: Christefano-oldaccount commentedI can confirm that it doesn't work in IE 6.
Comment #3
peterpoe CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: bsuttis commentedFrom my limited testing, it appears to work correctly in IE6/7 and Firefox 2.
Comment #9
peterpoe CreditAttribution: peterpoe commentedNo more reported errors in 2 weeks. I'll mark this as fixed.
Comment #10
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #11
patchak CreditAttribution: patchak commentedHey there, was this added to a new release of the module?? We should use the dev version??
Comment #12
patchak CreditAttribution: patchak commentedre-opening to have more infos about this....
Comment #13
peterpoe CreditAttribution: peterpoe commentedJep, the patch is in the latest dev. Re-closing.