Document $page['navigation'] in page.tpl [#1258280]

I'm looking at Zen page.tpl.php dated 25 April 2011.

At the top of the page, it says:

 * Regions:
 * - $page['help']: Dynamic help text, mostly for admin pages.
 * - $page['highlighted']: Items for the highlighted content region.
 * - $page['content']: The main content of the current page.
 * - $page['sidebar_first']: Items for the first sidebar.
 * - $page['sidebar_second']: Items for the second sidebar.
 * - $page['header']: Items for the header region.
 * - $page['footer']: Items for the footer region.
 * - $page['bottom']: Items to appear at the bottom of the page below the footer.

Should there be a line for $page['navigation'] as well? That seems to be the only similar thing that doesn't have one.

Comment	File	Size	Author
#1	zen-document-navigation-region-1258280-1.patch	816 bytes	barraponto

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

barraponto CreditAttribution: barraponto commented 5 September 2011 at 05:58

File	Size
zen-document-navigation-region-1258280-1.patch	816 bytes

That's because John Albin copied it from core. We can do better, let's add documentation for the navigation region.

Comment #2

barraponto CreditAttribution: barraponto commented 5 September 2011 at 05:58

Status:

Active

» Needs review

Comment #3

JohnAlbin

he/they/他

English

Taipei, Taiwan

CreditAttribution: JohnAlbin commented 8 September 2011 at 07:14

Title:	In page.tpl.php, $page['navigation'] exists, but isn't listed as a region in the documentation at the top.	» Document $page['navigation'] in page.tpl
Status:	Needs review	» Fixed

Thanks for reporting this, Jennifer!

And thanks for the patch, Capi!