WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3620 closed enhancement (fixed)

Accessibility update to login form

Reported by: Viper007Bond Owned by: ryan
Milestone: 2.2 Priority: high
Severity: minor Version: 2.1
Component: Administration Keywords: has-patch commit
Focuses: Cc:

Description

Was unaware that text readers used CSS, so I hid some stuff via display: none; rather than a large negative text indent.

Attached patch put the text indent back to how it was before.

Attachments (1)

login_accessibility.patch (1.1 KB) - added by Viper007Bond 7 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Viper007Bond7 years ago

  • Milestone changed from 2.1 to 2.1.1

Too late for 2.1.

comment:2 Viper007Bond7 years ago

  • Milestone changed from 2.1.1 to 2.2
[Sat 22:55:07] <MarkJaquith> well, 2.2 first, then backport

comment:3 ryan7 years ago

  • Owner changed from anonymous to ryan

comment:4 rob1n7 years ago

Patch looks good. Using text-indent instead of display:none is supposed to be the accessible "right thing to do."

comment:5 ryan7 years ago

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

(In [5064]) login accessibility patch from Viper007Bond. fixes #3620

comment:6 JeremyVisser7 years ago

At first, I thought something was broken, as my blog title started appearing over the top of where the WordPress logo, but realised it was because the old CSS was cached.

Don't forget to hit Ctrl+F5 if this happens to anyone else.

Note: See TracTickets for help on using tickets.