Contrib solution for non-collapsible fieldsets and missing titles

The same thing happend with the seven theme. We are currently waiting for http://drupal.org/node/1015798 to be resolved and backported to D7.

Hi Verve8media,

We could not wait for #1015798: Fieldsets inside vertical tabs have no title and can't be collapsed so we added our own css file to fix this temporarily in #1035398: Vertical tabs in node type form are wrong. As you have rubik theme installed, would you like to try modifying the scheduler.css file to make it work for you? Then there might be a more general solution we could incorporate.

Jonathan

The patch in #8 solves the problem for me with Rubik and Seven.

Yes, I've not used Rubik before but just checked and adding vertical-tabs-pane(s) qualifiers does improve things so that now you can actually see the fieldset legend. But with Rubik I think there is some more work to be done, because when the fieldset is opened the legend overlaps the conent. See screen shot attached.

If it worked fine for you, then maybe you have modified your own rubik css and set some margin/padding so that it works. My test was on Firefox 3.6 on mac, so it could be a browser thing. Could you post a screen shot too?

Jonathan

There is indeed stil a problem in Firefox. Chrome and Safari display the fields correct. (also on mac)

Subscribing, the patch in #8 seems to work.

Here is an updated patch which fixes the Rubik theme on Firefox. I had to use a slightly new tactic because this particular theme could not be fixed with general css which did not mess up other themes. So what the code now does is add the theme name as an extra class to the top-level div. This gives the ability to fix certain themes without affecting other themes, by starting the css with div.vertical-tabs.rubik for example. I;m particularly pleased with this (he said modestly) as I used a couple of lines of jquery to add the required class. ie

  var theme = Drupal.settings.ajaxPageState['theme'];
  $("div.vertical-tabs").addClass(theme);

So now, we can fix each default theme correctly, independently, and also add any fixes to other theme problems as they are reported.

Jonathan

I'd forgotten to include the jquery in the patch.

Added more detail to title, to assist those searching for this fix.
The latest code works correctly in Firefox, Safari and Opera (all on Mac).

#14 resolved the issue for me (chrome dev mac)

I think this issue is actually a Tao issue. Tao is Rubik's base theme and the theming bug really should be fixed there: #1212314: Fieldgroup legend tags within vertical tabs panes are hidden.

This is actually an issue in core: #1015798: Fieldsets inside vertical tabs have no title and can't be collapsed

Thanks. Yes we know, issue 1015798 has already been linked twice three times before in this thread ;-)

All of these css issues can be fixed in the base theme or in core, but we could not wait for those to be implemented so I devised a way that scheduler users could continue to use our D7 version in the meantime.

Thanks for you input.

Jonathan

For anyone wishing to use this in their modules, here are the two files which make up the solution. This is easier for you than trying to pick out the changes from the patches above. The css file can be extended to solve the problem for other themes not covered yet, and because it uses the theme name in the selector you can be confident that you won't mess up another theme by accident.

You will also nee to add something like this in the .module file:

function scheduler_form_node_type_form_alter(&$form, $form_state) {

  // Add the module css file
  drupal_add_css(drupal_get_path('module', 'scheduler') .'/scheduler.css');

  // Define the node settings and include the jquery file.
  $form['scheduler'] = array(
    '#type' => 'fieldset',
    '#title' => t('Scheduler settings'),
    '#weight' => 35,
    '#group' => 'additional_settings',
    '#attached' => array(
      'js' => array(
        'vertical-tabs' => drupal_get_path('module', 'scheduler') . "/scheduler_vertical_tabs.js"
      ),
    ),
  );

This was just how I did it, I'm sure there are other ways to add the jquery file, if the above is not suitable.

Jonathan

Core issue #1015798: Fieldsets inside vertical tabs have no title and can't be collapsed has just been committed to D7 dev. May need to check that it does not conflict with our solution.

In drupal 7.14 the error is not fixed. I've attached patch, which you can apply not only from module directory.

Hi,
The actual contents of the patch you have given in #21 seem to be indentical to mine in #14. The only difference is the way it's been generated - see image. Is that what you meant?

Jonathan

#21, patches need to target the module within the folder. Assumptions cannot be made about someone's folder structure (i.e. the scheduler module could be in someones profile/my_profile/modules/scheduler folder, etc.)

Also, the Rubik specific rules looked kind of wonky, so I've just removed them. Attached is said patch.

Before: http://cl.ly/image/2y0G0h0j3A0u
After: http://cl.ly/image/3s0T1J3X383j

I'm upping this issue to 'major', as it renders this module unusable for many themes.

#20, 7.15 fixes this issue, but the patch does not seem to conflict with it in any way, so I think it should still be committed for those who're still on older versions.

EDIT: Scratch that, it's functional but still lacks the fieldset titles: http://cl.ly/image/1U0e0u3T1100

Removed whitespaces.
Patch seems to work.

I applied the patch in #23 and it works beautifully for:

Drupal 7.17
scheduler-7.x-1.0
Tao 7.x-3.0-beta4
Rubik 7.x-4.0-beta8

Thanks to all.

D7 core has been improved, so this is no longer 'critical' but this patch is still required for Tao and Rubik themes.
The patch in #24 is fine, so I have marked it RTBC.

Thanks everyone for your patience and diligence. This is committed.
http://drupalcode.org/project/scheduler.git/commit/2f7ec59

We may have to roll this back in the future if further enhancements to D7 fix this in core. However, we will address this here for now.

I've checked this and it works as expected. Thanks Rick.

This might not be needed in Drupal 8 any more. I created a followup to investigate this: #2468819: Remove theming workarounds and delete scheduler.css.