I noticed a bug in four seasons which leads to disappearing menu buttons in the IMCE File Browser.

I narrowed it down to a div/fieldset problem. When I delete the fieldset/div part in templates.php, IMCE shows up fine. When I place it back, it bugs again.

This is a comparison betweeen the two files (left bad, right good):
comparison
It's because of this code:

/**
 * Override of theme_fieldset().
 * We use divs instead of fieldsets for better theming.
 */
function fourseasons_fieldset($variables) {
	$_path = drupal_strtolower(drupal_get_path_alias($_GET['q']));
	//if(!drupal_match_path($_path, "admin/appearance/settings/fourseasons")) {
  $element = $variables['element'];
  element_set_attributes($element, array('id'));
  _form_set_class($element, array('form-wrapper'));
  
  
 //
  $output = '<div' . drupal_attributes($element['#attributes']) . '>';
  if (!empty($element['#title'])) {
    $output .= '<div class="pseudo-fieldset-title">'. $element['#title'] .'</div>';
  }
   $output .= '<div class="pseudo-fieldset-content">';
  if (!empty($element['#description'])) {
    $output .= '<div class="fieldset-description">' . $element['#description'] . '</div>';
  }
  $output .= $element['#children'];
  if (isset($element['#value'])) {
    $output .= $element['#value'];
  }
  $output .= '</div>';
  $output .= "</div>\n";
  return $output;
 //}
}

//*/

You see that the code replaces also the 'div' to 'fieldset' in the IMCE code, which is not good.
I deleted the whole part in templates.php and everything works, but it weould be nice to have a cleaner fix then just delete the part. I just don't have that much knowledge about drupal and this theme, so maybe somebody else could help?

CommentFileSizeAuthor
comparison.JPG428.97 KBfarris-1
Support from Acquia helps fund testing for Drupal Acquia logo