Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
page-top-bottom-consistent.patch | 2.09 KB | Gábor Hojtsy | |
Comments
Comment #1
tic2000 CreditAttribution: tic2000 commentedI think no markup is the way to go.
Patch looks good to go.
Comment #2
sunyes, we can remove that.
Comment #3
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!