I noticed in page.tpl.php that the tabs wrapper div doesn't get closed. I've made the following changes - which may also improve other layout issues people might have:

line 68 (new line): <?php if ($tabs && !$tabs2): print '</div>'; endif; ?>
line 69:                 <?php if ($tabs2): print '<ul class="tabs secondary">'. $tabs2 .'</ul></div>'; endif; ?>

Obviously, you could combine line 68 and 69 into and if/else statement. The change to line 69 is the closing </div> Hope that helps.

Comments

andregriffin’s picture

andregriffin commented

Hmm, I'm not seeing how it's not being closed. Correct me if I'm wrong, but the current version of framework is built like so:

if ($tabs): print '<div id="tabs-wrapper" class="clear-block"><ul class="tabs primary">'. $tabs .'</ul>'; endif;
if ($tabs2): print '<ul class="tabs secondary">'. $tabs2 .'</ul>'; endif;
if ($tabs): print '<span class="clear"></span></div>'; endif;

There is only one div, the tabs-wrapper div and it is closed at the end of the clear span.

avr’s picture

avr commented

You're right. I must have deleted that line when I removed the "clear spans." Instead, I'm using a class of "clearfix" on the tabs wrapper.

Sorry for the confusion.

1 other thing I noticed and updated in my template.php file. Currently, if there are no sidebars, the "admin" class has no way of being added to the body selector. There is another thread with others discussing table layout. Making sure the admin class is added all the way around solved any table layout issues I was having.

Aaron

andregriffin’s picture

andregriffin commented
Status: Active » Closed (fixed)