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