WordPress.org

Make WordPress Core

Opened 7 months ago

Last modified 6 months ago

#42749 assigned enhancement

Accessibility enhancements for change password field

Reported by: alexstine Owned by: afercia
Milestone: 5.0 Priority: normal
Severity: normal Version: 4.9
Component: Users Keywords: has-patch
Focuses: accessibility Cc:

Description

Attaching accessibility enhancements to the users.js file which includes support for the change password box. The button is now never hidden and will offer a more accessible experience.

  • Announce when form is open or closed.
  • Focus in form after open.

Hope it's useful.

Attachments (3)

user-profile.js (13.2 KB) - added by alexstine 7 months ago.
Accessibility fixes for users.js.
42749.patch (3.2 KB) - added by alexstine 7 months ago.
users.js
42749-2.patch (3.2 KB) - added by alexstine 7 months ago.
Patch #2. Remove focus on password field.

Download all attachments as: .zip

Change History (11)

@alexstine
7 months ago

Accessibility fixes for users.js.

#1 follow-up: @SergeyBiryukov
7 months ago

  • Keywords needs-patch added

Hi @alexstine, welcome to WordPress Trac! Thanks for the ticket.

Could you create a patch with your changes? Working With Patches tutorial might be helpful.

@alexstine
7 months ago

users.js

#2 in reply to: ↑ 1 @alexstine
7 months ago

  • Focuses multisite removed

I've attached a patch file.

Replying to SergeyBiryukov:

Hi @alexstine, welcome to WordPress Trac! Thanks for the ticket.

Could you create a patch with your changes? Working With Patches tutorial might be helpful.

#3 @SergeyBiryukov
7 months ago

  • Keywords has-patch added; needs-patch removed

#4 @afercia
7 months ago

  • Owner set to afercia
  • Status changed from new to assigned

#5 @alexstine
7 months ago

I have made another patch to exclude focus. I'm attaching it.

@alexstine
7 months ago

Patch #2. Remove focus on password field.

#6 @afercia
6 months ago

I'd second to not move focus programmatically, as it is often just an assumption on a specific workflow. See also #42852 where the attached patch incorporates some of the changes proposed here.

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


6 months ago

#8 @afercia
6 months ago

  • Milestone changed from Awaiting Review to 5.0
Note: See TracTickets for help on using tickets.