The Seven theme moves both the primary and secondary tabs into the header

[avpaderno]

The Seven themes uses the following hook, which moves primary and secondary tabs into the header section.

function seven_preprocess_layout(&$variables) {
  // Don't modify layouts that are being edited.
  if (!$variables['admin']) {
    // Move the page title and tabs into the "header" area, to fit with Seven's
    // markup requirements.
    if (isset($variables['content']['header'])) {
      if ($variables['title']) {
        $title = '<h1 class="page-title">' . $variables['title'] . '</h1>';
        $variables['content']['header'] .= $title;
        $variables['title'] = NULL;
      }
      if ($variables['tabs']) {
        $tabs = '<div class="tabs">' . $variables['tabs'] . '</div>';
        $variables['content']['header'] .= $tabs;
        $variables['tabs'] = NULL;
      }
    }
  }

The effect is that tabs in pages like /admin/content/comment appear as in the following screenshot.

The tabs should be instead rendered as in the following screenshot.

When there are just primary tabs, all is fine.

(I fixed the second screenshot, which was not the correct one.)

[avpaderno]

layout_preprocess_layout() uses the following code.

if (!$block_exists['tabs']) {
  $variables['tabs'] = menu_local_tabs();
}

menu_local_tabs() returns primary and secondary tabs already rendered. Splitting them requires splitting a string in two.

[avpaderno]

Actually, it is sufficient to use the following code, in seven_preprocess_layout().

    // Move the page title and primary tabs into the "header" area.
    if (isset($variables['content']['header'])) {
      if ($variables['title']) {
        $title = '<h1 class="page-title">' . $variables['title'] . '</h1>';
        $variables['content']['header'] .= $title;
        $variables['title'] = NULL;
      }

      if ($variables['tabs']) {
        $primary_tabs = theme('menu_local_tasks', array(
          'primary' => menu_primary_local_tasks(),
        ));
        $variables['content']['header'] .= $primary_tabs;

        $secondary_tabs = theme('menu_local_tasks', array(
          'secondary' => menu_secondary_local_tasks(),
        ));
        $variables['tabs'] = $secondary_tabs;
      }
    }

The full code of the hook would become the following one.

function seven_preprocess_layout(&$variables) {
  // Don't modify layouts that are being edited.
  if (!$variables['admin']) {
    // Move the page title and tabs into the "header" are.
    if (isset($variables['content']['header'])) {
      if ($variables['title']) {
        $title = '<h1 class="page-title">' . $variables['title'] . '</h1>';
        $variables['content']['header'] .= $title;
        $variables['title'] = NULL;
      }

      if ($variables['tabs']) {
        $primary_tabs = theme('menu_local_tasks', array(
          'primary' => menu_primary_local_tasks(),
        ));
        $variables['content']['header'] .= $primary_tabs;

        $secondary_tabs = theme('menu_local_tasks', array(
          'secondary' => menu_secondary_local_tasks(),
        ));
        $variables['tabs'] = $secondary_tabs;
      }
    }
  }
}

This is how the tabs would appear.

[avpaderno]

When there are no secondary tabs, everything is rendered as expected.

[avpaderno]

[olafgrabienski]

Thanks for the issue and the fix! The PR works for me in the sandbox. I've tested the following pages:

• admin/content/comment
• admin/structure/types/manage/post/display

[argiepiano]

Thanks @avpaderno.

But... the issue with this fix is that now the system messages in the default admin layout appear in between the primary and the secondary tabs. This, in my opinion, is less than ideal. Isn't there a way to fix this minor problem with css?

With the patch:

Without: