Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#34943 closed defect (bug) (fixed)

Remove title attributes: wp-login.php

Reported by: afercia's profile afercia Owned by: afercia's profile afercia
Milestone: 4.5 Priority: normal
Severity: normal Version: 4.4
Component: Login and Registration Keywords: has-patch, commit, title-attribute
Focuses: ui, accessibility Cc:

Description

See #24766 for some background. In the login screen, there are a couple of title attributes:

https://cldup.com/H3JcnVjb6Q.png

They serve no useful purpose and add noise for screen reader users.

Also, discussed a bit with the Italian polyglots team, they're difficult to translate. While in English there's (maybe?) a subtle irony in a sentence like "Password Lost and Found", the current Italian translation is "Ufficio oggetti smarriti - Password" which sounds a bit weird. I guess it's not so easy to translate the tone of this sentence in other languages too.

I'm not opposed to a bit of irony and in core there are some examples of irony I appreciate ("Slow down cowboy ... ") :) but in this case I'd propose to just remove these 2 title attributes.

Attachments (1)

34943.diff (1.4 KB) - added by afercia 9 years ago.

Download all attachments as: .zip

Change History (6)

@afercia
9 years ago

#1 @afercia
9 years ago

  • Keywords has-patch added
  • Owner set to afercia
  • Status changed from new to assigned

#2 @SergeyBiryukov
9 years ago

  • Keywords commit added

Agreed.

#3 @afercia
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 35846:

Accessibility: remove no-purpose title attributes from the login screen.

Also, it's hard to convey the ironic tone of the sentences used for these title attributes in languages other than English.

Fixes #34943.

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


9 years ago

#5 @afercia
8 years ago

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