Hey I dont know if this had to do with a preconfigured setup that i had or something but when I enabled the theme the tabs for each page were overlaying on the messages . see attached file.

I cant image that you would have created the theme like this so I figured it might be something on my side. My drupal install is standard and i havent started any major tweeks to anything. Using firefox 3.6.12

I solved the problem by adding

div.tabs {
display:inline-block; /*this code*/
margin:0 0 5px;
}

in the tabs.css

--i like this theme alot!!

CommentFileSizeAuthor
#1 zen_ninesixty.tab_overlays_messages.patch676 bytesAnonymous (not verified)
tabsOverlay.gif45.43 KBAnonymous (not verified)

Comments

Anonymous’s picture

StatusFileSize
new676 bytes

I spotted this also. The better way to resolve is to override theme_menu_local_tasks(). Add this to your template.php (the one in zen_ninesixty/ folder):

 /**
  * Overrides theme_menu_local_tasks().
  */
function zen_ninesixty_menu_local_tasks() {
  $output = '';

  if ($primary = menu_primary_local_tasks()) {
    $output .= "<ul class=\"tabs primary clear-block\">\n" . $primary . "</ul>\n";
  }

  if ($secondary = menu_secondary_local_tasks()) {
    $output .= "<ul class=\"tabs secondary clear-block\">\n" . $secondary . "</ul>\n";
  }

  return $output;
}

Alternatively, you can apply the patch from http://drupal.org/node/984796 and them apply the attached patch.

Anonymous’s picture

Priority: Minor » Critical
Status: Active » Needs review
Anonymous’s picture

that looks a bit heavy just to get the tabs in the right place. I dont know if i want to add php just to get that right, it should by default be right in the css.

Robrecht Jacques’s picture

Status: Needs review » Reviewed & tested by the community

I can confirm that the tabs show up correctly after applying the patch.

Just like "clearfix" has been replaced by "clear-block" in node.tpl.php and other files, it should be replaced in the tabs as well.