Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#24114 closed enhancement (fixed)

Twenty Thirteen: Remove hgroup

Reported by: obenland's profile obenland Owned by: lancewillett's profile lancewillett
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

With the removal of the <hrgroup> element from the spec, let's remove it in Twenty Thirteen too.

I'm pretty sure we'd blow back compat if we'd attempt to remove it in older themes.

Attachments (3)

24114.diff (3.0 KB) - added by obenland 11 years ago.
24114.1.diff (4.5 KB) - added by lancewillett 11 years ago.
24114.2.diff (4.3 KB) - added by lancewillett 11 years ago.

Download all attachments as: .zip

Change History (12)

@obenland
11 years ago

#1 follow-up: @lancewillett
11 years ago

With the removal of the <hrgroup> element from the spec

hgroup? Can you please link to discussion and relevant docs for the removal?

#3 @lancewillett
11 years ago

Can we get rid of the extra div element? Seems like we could apply the same styles to the anchor element, since it's a block-level element and always present.

#4 @lancewillett
11 years ago

.1 removes the entire element (hgroup or div) and moves CSS and JS references to class value of "masthead" instead.

#5 follow-up: @philiparthurmoore
11 years ago

How do you plan on dealing with the hgroup cruft in html5.js?

#6 @philiparthurmoore
11 years ago

  • Cc philip@… added

#7 in reply to: ↑ 5 @lancewillett
11 years ago

Replying to philiparthurmoore:

How do you plan on dealing with the hgroup cruft in html5.js?

Leaving it for now, will push an update once the author has a new version. It's very minor, in my opinion.

#8 @lancewillett
11 years ago

.2 updates to use home-link as class value instead of moving masthead.

#9 @lancewillett
11 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 24072:

Twenty Thirteen: remove hgroup from the theme markup structure since it's been dropped from HTML5 spec. Props obenland for the starter patch. Fixes #24114.

Note: See TracTickets for help on using tickets.