Opened 10 years ago
Closed 10 years ago
#24522 closed defect (bug) (fixed)
Remove the cite element from comment_html5()
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 3.6 |
Component: | Comments | Keywords: | has-patch |
Focuses: | Cc: |
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
@
10 years ago
- Summary changed from Remove the cite element from comment_html5 to Remove the cite element from comment_html5()
#4
follow-up:
↓ 5
@
10 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
@
10 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
cite
in 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
.
Related: #23128/[23716]