IE7 does not show tables in collapsed fieldsets

This is probably an issue with all versions of Drupal.

subscribe

As the original poster noted, if the fieldset is expanded by default, the toggle mechanism works. I've used this as a workaround:

$form = array(
  '#title' => t('Fieldset Title'),
  '#collapsible' => TRUE,
  '#collapsed' => (strpos($_SERVER['HTTP_USER_AGENT'],'MSIE 7.0') !== FALSE) ? FALSE : TRUE,  // default to expanded for IE7
  '#type' => 'fieldset',
  '#attributes' => array('id' => 'fieldset_unique_id'),  // let's make it easy for jquery to find our fieldset
);

if (strpos($_SERVER['HTTP_USER_AGENT'],'MSIE 7.0') !== FALSE) {
  // collapse the fieldset when the page is ready and the toggle works.
  $js_code = "
    if (Drupal.jsEnabled) {
      $(document).ready(function() {
        $('#fieldset_unique_id').addClass('collapsed');
      });
    };
  ";
  drupal_add_js($js_code,'inline','footer');
}

This problem is plaguing some of my users. We have a Drupal v5.14 setup. Any patch or workaround until this bug can big fixed would be appreciated as it renders the ability to post to Drupal for IE7 impossible if any default settings in a non-working fieldset require changes.

(The suggested workaround in #4 is impractical as it would, as I understand it, require patching every module that creates a collapsed fieldset. If there is a way to fix this by changing only one or two files I don't get that from the the post.)

It looks like IE7 is somewhat going backwards...

In the system.css there are a few specific lines for IE6 (using * html):

/* Note: IE-only fix due to '* html' (breaks Konqueror otherwise). */
*html.js fieldset.collapsed legend,
*html.js fieldset.collapsed legend *,
*html.js fieldset.collapsed table * {
display: inline;
}

The symptom here is that tables aren't being displayed when coming from collapsed to open. Adding this fixes it:

html.js fieldset.collapsed table * {
display: inline;
}

Of course, the system.css states that this will break Konqueror, and core doesn't have an IE7 style sheet (although themes do), so I am unsure of where it could go.

Hopefully this will help people who are in need of a quick fix and also help to find a permanent solution! :-)

Jack

Thanks, jdipper. I tried this on v5.15 but it did not seem to have any effect. Logging in on my system as one of the users the collapsed fieldsets were still collapsed and could not be opened.

Yeah, should have said that I was using 6.9 :-\

I do think, however, that it's a related issue; seems IE's having issues with assigning styles when classes are assigned by JS (although I have no idea why it's only affecting tables).

Do you have access to IE6 and see if it works on that? If it does, it might be worth finding out if any *html entries can be used (as IE7 doesn't use them, you'll have to unstar them to test it out).

Failing that I'll take a peak this weekend to see if I can spot why IE7 isn't playing ball with D5.

Jack

I don't have access to IE6 but I have IE7. I tried it with IE7 and unstarred the entries. Fieldsets still collapsed.

subscribed

Subscribing. This affects my Term Permissions module. The suggested hack in #4 works fine for me, though obviously isn't a long-term solution.

Solution from #7 works fine at our site for IE7 and Drupal 6.9.

This could be the issue: http://cssbeauty.com/skillshare/discussion/1515/fieldset-woe-in-safari-ie7/ , a drupalized way to solve that would be in fieldsets with tables, use #prefix => div for the first element, #suffix => div for the last element, or to do things more properly, tweak the redering process.

Whatever IE is... it's definitely not a cause for us to treat this as critical.

This is a usability issue not just an IE issue. If users cannot access the features of Drupal that seems to me to be fairly important.

This is a usability issue not just an IE issue. If users cannot access the features of Drupal and post that seems to me to be fairly important.

Note to future cross browser battlers:

Fusion sub theme, ubercart checkout page, Terms of Service Ubercart Module, IE7, Internet Explorer 7 CSS Problems.

To cure the inline table on payment details, I had to resort to:

#payment_details div   {
	height: 200px;
}
#payment_details div div {
	height: auto;
}

I gave up on a scrolling terms of service node. The node is embedded, and fusion has too much (in this case) style hooked up to the 3 or so containing divs:

.tos-text {
	height: auto; /* epic fail.. give up on usability */	
}

But at least it aint IE6 anymore!

Subscribe

Drupal 7 require Internet Explorer 8.x and later, see https://www.drupal.org/node/61509 :)

That page was misleading (and looks like it since has been fixed). We haven't officially dropped support for IE7 in Drupal 7. Certainly these issues are very low priority now, but if someone wants to fix them and can do it in a way that doesn't hurt other browsers, it's still possible to do so.