[wp-trac] [WordPress Trac] #30057: Twenty Fifteen: Site description is not a heading, replace the h2 element for a p

[WordPress Trac]

#30057: Twenty Fifteen: Site description is not a heading, replace the h2 element
for a p
-------------------------------------+------------------------------
 Reporter:  rianrietveld             |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  Bundled Theme            |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  accessibility
-------------------------------------+------------------------------

Comment (by georgestephanis):

 We've been rather inconsistent to date in using h1/h2 for site title/site
 description --

 * TwentyTen used a div - https://core.trac.wordpress.org/browser/trunk/src
 /wp-content/themes/twentyten/header.php#L68
 * TwentyEleven used h1/h2 in an hgroup -
 https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentyeleven/header.php#L75
 * TwentyTwelve used h1/h2 in an hgroup -
 https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentytwelve/header.php#L39
 * TwentyThirteen used h1/h2 in an hgroup -
 https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentythirteen/header.php#L38
 * TwentyFourteen used a h2 in sidebar.php -
 https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentyfourteen/sidebar.php#L15
 * TwentyFifteen now uses the same as eleven-thirteen -- h1/h2 in an hgroup
 - https://core.trac.wordpress.org/browser/trunk/src/wp-
 content/themes/twentyfifteen/header.php#L33

 As long as it's clustered in an hgroup, it should be fine semantically,
 imho.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/30057#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform