Add heading before local tasks for screen-reader users

Seven builds its own variables for primary and secondary tasks, bypassing the theme function. This won't work in Garland either. Its a bit weak to place this in the theme function since many themes split the tabs variable similar to how Seven and Garland are doing it, unfortunately I don't have a better solution in my brain right now.

Sure, both do it in template.php.

Seven does this in seven_preprocess_page():

/**
 * Override or insert variables into the page template.
 */
function seven_preprocess_page(&$vars) {
  $vars['primary_local_tasks'] = menu_primary_local_tasks();
  $vars['secondary_local_tasks'] = menu_secondary_local_tasks();
}

Garland on the other hand overrides the theme function:

/**
 * Returns the rendered local tasks. The default implementation renders
 * them as tabs. Overridden to split the secondary tasks.
 */
function garland_menu_local_tasks() {
  return menu_primary_local_tasks();
}

For both it might just be easier to put the headers in page.tpl.php?

I think it's going to have to be in the template.php because of the translation of the hidden header, t('Primary Tasks') & t('Secondary Tasks') can't be run from the tpl.php files, right.

@Everett, are you going to re-roll it?

t() can be run from anywhere and I'm pretty sure you can extract translatable strings from tpl files - best to ask someone in the locale team maybe (Gabor?).

Ok, I think the problem is really that this hasn't been added for Seven. I think that headings are there in the other themes.

$ grep 'Secondary' bartik/templates/page.tpl.php 
 * - $secondary_menu (array): An array containing the Secondary menu links for
            'text' => t('Secondary menu'),
$ grep 'Secondary' garland/template.php 
        'text' => t('Secondary menu'),
$ grep -i 'Secondary' seven/

Patch for Seven is included. And Jeff, you were definitely right about t(). Not sure where I got that idea from that it wouldn't work there.

#9: 867114-9.patch queued for re-testing.

Yeah, its totally the wrong patch, lol, will take a look since Mikes gone on holiday.

Ok.. Maybe I've uploaded the right one this time..

Just referencing an issue with Garlands local tasks: #903814: Some admin pages not displaying in the Overlay in Garland

This looks pretty good to me. It applies nicely. I've added a tag for Overlay so hopefully someone there looks at this & can help.

I just tested this with Bartik, Stark, Garland and Seven as admin themes, and it's all good.

I'm not sure why the testbot doesn't like Everett's patch. I've attached a straight re-roll of it to see if the bot likes it any better.

Looks good to me. Jacine, thanks for the re-roll. The overlay patch that Everett introduced works fine too.

Committed to CVS HEAD. Thanks.