Make WordPress Core

Opened 8 months ago

Last modified 9 days ago

#42537 new defect (bug)

Remove title attributes: wp-login.php

Reported by: SergeyBiryukov Owned by:
Milestone: 5.0 Priority: normal
Severity: normal Version: 4.9
Component: Login and Registration Keywords: has-patch title-attribute
Focuses: accessibility Cc:


Background: #34625

On Multisite, the title attribute in login_header() is different from the link text and has some meaning.

<a href="http://ms.wordpress.dev/" title="Network Title" tabindex="-1">Site Title</a>

On single site, however, the title attribute just duplicates the link text and is redundant:

<a href="http://wordpress.dev/" title="Powered by WordPress" tabindex="-1">Powered by WordPress</a>

Most of the title attributes are already removed from core via #24766 and related tickets. We should consider removing this one as well, either completely or only if it duplicates the link text.

The login_headertitle filter could either be deprecated or repurposed to filter the link text instead.

Attachments (1)

42537.diff (612 bytes) - added by chetan200891 8 months ago.
Created patch to remove title attribute.

Download all attachments as: .zip

Change History (5)

8 months ago

Created patch to remove title attribute.

#1 @chetan200891
8 months ago

  • Keywords has-patch added

I've attached the patch 42537.diff.

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.

8 months ago

#3 @afercia
8 months ago

  • Milestone changed from Awaiting Review to 5.0

See also #42632

#4 @afercia
9 days ago

  • Keywords title-attribute added
Note: See TracTickets for help on using tickets.