While writing docs for the introduction of $page_top and $page_bottom, I've stumbled on issues with how the documentation was not updated in system module's page.tpl.php, and how $page_top is not consistent with its wrapper markup to $page_bottom (which lacks any wrapper markup). We should either have suggested / default wrapper markup for both of these or none of these. The toolbar works either way.

CommentFileSizeAuthor
page-top-bottom-consistent.patch2.09 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tic2000’s picture

I think no markup is the way to go.
Patch looks good to go.

sun’s picture

Status: Needs review » Reviewed & tested by the community

yes, we can remove that.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.