Patch #19339
Put news articles into <article> tags
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | % 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.
Associated revisions
Put news articles into <article> tags (#19339).
Patch by An Phan.
History
#1
Updated by Toshi MARUYAMA over 7 years ago
- Target version set to 3.1.0
#2
Updated by Jean-Philippe Lang over 7 years ago
Why not doing the same on the news detail page, eg. /news/1?
#3
Updated by An Phan over 7 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 about 7 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.