Opened 5 years ago
Last modified 4 years ago
#50202 new defect (bug)
Use <strong> tag instead of <b> tag in `Walker_Comment::html5_comment`
Reported by: | itowhid06 | Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | needs-patch |
Focuses: | accessibility | Cc: |
Description
It is well documented that when some text is bolded using the <b> tag, it will come to no benefit for users using screen readers. This patch aims to fix that in an instance of html5_comment
method in Walker_Comment
.
Attachments (1)
Change History (4)
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
5 years ago
#3
@
5 years ago
- Keywords needs-patch added; has-patch close 2nd-opinion removed
- Milestone changed from Awaiting Review to Future Release
- Version trunk deleted
This ticket was discussed during today's accessibility bug-scrub.
While it's true the usage of a <b>
element in this case is not incorrect, it is also true that right now the commenter username has no specific semantics. For example, screen readers would read it as normal text. It would be nice to add some semantic, not sure a <strong>
element would be appropriate though. Also, there are backwards compatibility concerns. Moving to future release for now, pending some research and investigation.
Thanks for the patch! The
<b>
tag was introduced in [24539] / #24522 and is only used here as a wrapper for comment author name, not to signify importance.Per the HTML standard, this appears to be valid usage:
Per MDN web docs, too: