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?

CommentFileSizeAuthor
#7 conditional_fields.js IE patch3.62 KBpeterpoe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bsuttis’s picture

Version: 5.x-2.x-dev » 5.x-1.x-dev

Version was wrong =/

Christefano-oldaccount’s picture

Priority: Normal » Critical

I can confirm that it doesn't work in IE 6.

peterpoe’s picture

I 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)?

Christefano-oldaccount’s picture

The 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.

dragoff’s picture

Tested 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 :]

scottrigby’s picture

I'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!

peterpoe’s picture

Status: Active » Needs review
FileSize
3.62 KB

The 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?

bsuttis’s picture

From my limited testing, it appears to work correctly in IE6/7 and Firefox 2.

peterpoe’s picture

Status: Needs review » Fixed

No more reported errors in 2 weeks. I'll mark this as fixed.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

patchak’s picture

Hey there, was this added to a new release of the module?? We should use the dev version??

patchak’s picture

Status: Closed (fixed) » Active

re-opening to have more infos about this....

peterpoe’s picture

Status: Active » Closed (fixed)

Jep, the patch is in the latest dev. Re-closing.