Removed dangling pubdate attributes #93

Merged
merged 1 commit into from Dec 29, 2012

Conversation

Projects
None yet
3 participants
@kwight
Member

kwight commented Oct 11, 2012

The attribute "pubdate" appears to no longer be part of the HTML5 spec, so remove them (they weren't being used anyways)

Removed dangling pubdate attributes
The <time> attribute "pubdate" appears to no longer be part of the HTML5 spec, so remove them (they weren't being used anyways)
@lancewillett

This comment has been minimized.

Show comment Hide comment
@lancewillett

lancewillett Oct 11, 2012

Member

I vote to leave them as is, discussing in a core ticket also: http://core.trac.wordpress.org/ticket/22095

Member

lancewillett commented Oct 11, 2012

I vote to leave them as is, discussing in a core ticket also: http://core.trac.wordpress.org/ticket/22095

@mfields

This comment has been minimized.

Show comment Hide comment
@mfields

mfields Nov 10, 2012

Contributor

After reading the conclusion on the core ticket: http://core.trac.wordpress.org/ticket/22095 I think it may be best to follow core's example and drop the pubdate attribute from _s.

Contributor

mfields commented Nov 10, 2012

After reading the conclusion on the core ticket: http://core.trac.wordpress.org/ticket/22095 I think it may be best to follow core's example and drop the pubdate attribute from _s.

mfields added a commit that referenced this pull request Dec 29, 2012

Merge pull request #93 from kwight/patch-2
Remove invalid pubdate value from time elements in template-tags.php. See #93

@mfields mfields merged commit 10c87ae into Automattic:master Dec 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment