Twenty Twenty: Comments accessibility - links should be identifiable

[audrasjb]

In Twenty Twenty, the links displayed in the Comments section are not visually identifiable. We should make them different from regular text for better accessibility.

[audrasjb]

"Logged in" and "Log out?" should be visually identifiable as links.

I made links red and with underline on hover

Trac ticket: https://core.trac.wordpress.org/ticket/56269

[whaze]

hello,

I made a PR : i changed the color of the links in red and with underline on hover.

[audrasjb]

Thanks for the patch!
However I think it would be better for accessibility and consistency to use the same pattern as other content links: different color + underline and remove underline on hover (but keep the different color).

See screenshot below (comment’s content)

i made your recommanded change for accessibility

[sabernhardt]

Thanks!

I think we could remove those styles from .comment-notes links as well. Twenty Twenty itself does not add any links to that message, but a plugin or a child theme might.

[mukesh27]

Hi there!

Yes it is better to remove those CSS for .comment-notes and .logged-in-as.

.comment-respond .comment-notes a {
	color: inherit;
	text-decoration: none;
}

.comment-respond .comment-notes a:focus,
.comment-respond .comment-notes a:hover,
.comment-respond .logged-in-as a:focus,
.comment-respond .logged-in-as a:hover,
.comment-respond .comment-notes a:hover {
	text-decoration: underline;
}

i removed .comment-notes useless css

[sabernhardt]

"Logged in as" message with the patch

[sabernhardt]

hovering cursor over "Edit your profile" link

[audrasjb]

Thanks for testing! Self assigning for commit.

[audrasjb]

In 53797:

Twenty Twenty: Ensure comments section links are identifiable as links.

Links should look like links. This changeset makes sure that links displayed in the comments section of Twenty Twenty are identifiable as links.

Props audrasjb, whaze, sabernhardt, mukesh27.
Fixes #56269.

Committed in https://core.trac.wordpress.org/changeset/53797