WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#42749 closed enhancement (duplicate)

Accessibility enhancements for change password field

Reported by: alexstine's profile alexstine Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.9
Component: Users Keywords:
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 years ago.
Accessibility fixes for users.js.
42749.patch​ (3.2 KB) - added by alexstine 7 years ago.
users.js
42749-2.patch​ (3.2 KB) - added by alexstine 7 years ago.
Patch #2. Remove focus on password field.

Download all attachments as: .zip

Change History (18)

@alexstine
7 years ago

Accessibility fixes for users.js.

#1 follow-up: @SergeyBiryukov
7 years 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 years ago

users.js

#2 in reply to: ↑ 1 @alexstine
7 years 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 years ago

  • Keywords has-patch added; needs-patch removed

#4 @afercia
7 years ago

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

#5 @alexstine
7 years ago

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

@alexstine
7 years ago

Patch #2. Remove focus on password field.

#6 @afercia
7 years 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.
7 years ago

#8 @afercia
7 years ago

  • Milestone changed from Awaiting Review to 5.0

This ticket was mentioned in ​Slack in #accessibility by rianrietveld. ​View the logs.
6 years ago

#10 @afercia
6 years ago

  • Owner afercia deleted

#11 @pento
6 years ago

  • Milestone changed from 5.0 to 5.1

#12 @pento
6 years ago

  • Keywords needs-refresh added
  • Milestone changed from 5.1 to Future Release

Patch needs updating.

#13 @afercia
6 years ago

A refreshed patch is ready on the related ticket #42852

#14 @alexstine
4 years ago

  • Keywords has-patch needs-refresh removed
  • Resolution set to fixed
  • Status changed from assigned to closed

This has been implemented in another TRAC ticket. Closing.

#15 @desrosj
4 years ago

  • Milestone Future Release deleted
  • Resolution changed from fixed to duplicate

Duplicate of #42852.

Note: See TracTickets for help on using tickets.