[wp-trac] [WordPress Trac] #17611: TwentyEleven Theme H1 Priority

WordPress Trac wp-trac at lists.automattic.com
Thu Jun 23 21:41:48 UTC 2011


#17611: TwentyEleven Theme H1 Priority
------------------------------------+-----------------------
 Reporter:  emiluzelac              |       Owner:
     Type:  defect (bug)            |      Status:  reopened
 Priority:  normal                  |   Milestone:  3.2
Component:  Bundled Theme           |     Version:  3.2
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+-----------------------

Comment (by iandstewart):

 I've added an updated patch that
 * Reduces the heading level by 1 for posts that aren't is_singular, making
 the headings h2 for "indexed" posts
 * Makes the headings in nav and page-level aside elements h3
 * Leaves the site title an h1 element with the tagline an h2 on all pages
 inside an hgroup. To fully match Twenty Ten's heading pattern, with a
 dynamic heading for the site title, we should also use a dynamic heading
 for the first nav element on the page too which seems like too much for
 me.
 * Add a missing nav heading in image.php
 * Removes unneeded nav and heading in single.php

 There are still pages with multiple h1s. Archive pages will have 1 for the
 site title and 1 for the page title (Category Archives: X). The Showcase
 will have three h1s; the site title, the featured post heading, and the
 recent post heading. I don't see this as a problem though.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/17611#comment:67>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list