Project

General

Profile

Actions

Patch #19339

closed

Put news articles into <article> tags

Added by An Phan over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Category:
News
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

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


Files

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

Updated by Toshi MARUYAMA over 9 years ago

  • Target version set to 3.1.0
Actions #2

Updated by Jean-Philippe Lang over 9 years ago

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

Actions #3

Updated by An Phan over 9 years ago

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

Actions #4

Updated by Jean-Philippe Lang over 9 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.

Actions

Also available in: Atom PDF