I think it's wrong to name the id of the wrapper for content and sidebars to "page" semantics-wise. The page is the whole page template.

I propose two solutions:

  1. Rename page to main
  2. Rename page to main and add a wrapper around all content named page.
Members fund testing for the Drupal project. Drupal Association Learn more


ptrl’s picture

ptrl’s picture

patch tao-927578-rename-and-wrapper.patch missing newline at end of file

barraponto’s picture

Status: Active » Postponed

I like it, and if I were a mantainer, I'd go with #main for 7.x-4.x. I wouldn't like to break all of the child themes that expect a #page id in page.tpl.php. But truth is #main is the content+sidebars wrapper in system.module and Zen, and I'd expect it in Tao as well.

But no #page wrapper, it just doesn't make sense. Use the body tag.