Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#19093 closed defect (bug) (invalid)

Replace <time> with <data> in twentyeleven

Reported by: hd-J Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch 2nd-opinion
Focuses: Cc:


There was a change in the HTML5 draft today, leading to the removal of <time>, now replaced by <data> (see ticket here)

Today in twentyeleven we use <time pubdate datetime=""></time>, and the new draft suggests the following:
<data itemprop="datePublished" value=""></data>

While many (me included) do not necessarily agree with the changes, I would suggest the following patch.

I'd be happy to have a second opinion though.


Attachments (1)

timetodata.diff (2.7 KB) - added by hd-J 4 years ago.
Replace <time> by <data>

Download all attachments as: .zip

Change History (6)

4 years ago

Replace <time> by <data>

#1 @kawauso
4 years ago

  • Component changed from Themes to Bundled Theme

#2 @Otto42
4 years ago

Suggest waiting on this until the standard becomes more standard, since a lot of people are against the removal of <time>. Won't hurt to wait a few months.

#3 @nacin
4 years ago

Agreed with Otto42. Let's let it settle upstream.

#4 @toscho
4 years ago

  • Cc info@… added

Agree with Otto. Plus, this change could affect child themes. Their authors may need some time to prepare.

#5 @nacin
4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Well, that lasted long.


Feel free to re-open if it becomes valid again.

Note: See TracTickets for help on using tickets.