Patch #19339

Put news articles into <article> tags

Added by An Phan almost 3 years ago. Updated over 2 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:News
Target version:3.1.0

Description

The news article HTML should be wrapped in <article> tags. This follows HTML5 practices, and makes theme styling easier.

better-news-structure.diff Magnifier (1.34 KB) An Phan, 2015-03-11 07:18

Associated revisions

Revision 14308
Added by Jean-Philippe Lang over 2 years ago

Put news articles into <article> tags (#19339).

Patch by An Phan.

History

#1 Updated by Toshi MARUYAMA almost 3 years ago

  • Target version set to 3.1.0

#2 Updated by Jean-Philippe Lang almost 3 years ago

Why not doing the same on the news detail page, eg. /news/1?

#3 Updated by An Phan almost 3 years ago

We sure can, but is it necessary though? The news details are directly wrapped in the #main div itself.

#4 Updated by Jean-Philippe Lang over 2 years ago

  • Subject changed from Put news articles into, well, <article> tags to Put news articles into <article> tags
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Comitted in r14308, thanks.

Also available in: Atom PDF