Opened 13 years ago
Closed 13 years ago
#24522 closed defect (bug) (fixed)
Remove the cite element from comment_html5()
| Reported by: | ocean90 | Owned by: | ocean90 |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Comments | Version: | 3.6 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
http://wordpress.org/support/topic/comment-author-markup-in-html5-cite-is-correct
In HTML5 the <cite> element shouldn't be used, when referring to a name/author.
Sources:
- http://www.w3.org/TR/html5/text-level-semantics.html#the-cite-element
- http://html5doctor.com/blockquote-q-cite/#cite
The Release Candidate proposes the <b> element, which sounds fair enough. obenland mentioned, that it would also set itself apart from the sorrounding <span>.
Attachments (2)
Change History (9)
#3
@
13 years ago
- Summary Remove the cite element from comment_html5 → Remove the cite element from comment_html5()
#4
follow-up:
↓ 5
@
13 years ago
+1 to this change. A brief look at Twenty Thirteen, no changes needed as we don't style that particular element with a cite in any selectors.
#5
in reply to: ↑ 4
@
13 years ago
Replying to lancewillett:
+1 to this change. A brief look at Twenty Thirteen, no changes needed as we don't style that particular element with a
citein any selectors.
A side effect will be, that the font will be bold, which can be fixed by adding font-weight: normal to .comment-author .fn.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Related: #23128/[23716]