Make WordPress Core

Opened 7 months ago

Last modified 5 weeks ago

#60726 accepted defect (bug)

The WordPress core password reset needs to pre-populate the username to meet WCAG 2.2

Reported by: estelaris's profile estelaris Owned by: joedolson's profile joedolson
Milestone: 6.7 Priority: normal
Severity: normal Version:
Component: Login and Registration Keywords: has-patch changes-requested
Focuses: ui, accessibility Cc:

Description (last modified by sabernhardt)

According to new WCAG 2.2 success criterion for 3.3.7 redundant entry.

The criterion establishes that information previously entered by or provided to the user that is required to be entered again the same process is either:

  • auto-populated, or
  • available for the user to select

There are 3 exceptions:

  • re-entering the information is essential,
  • the information is required to ensure the security of the content, or
  • previously entered information is no longer valid.

Once the user has performed the process of requesting a new password, the redirected form should have the username filled-in to pass. As of now, this is the form that the user is redirected to:

Attachments (1)

wp-reset-password-redirect-form.png (15.7 KB) - added by estelaris 7 months ago.

Download all attachments as: .zip

Change History (18)

This ticket was mentioned in Slack in #design by estelaris. View the logs.


7 months ago

#2 @joedolson
7 months ago

  • Component changed from General to Login and Registration
  • Keywords needs-patch added; needs-design removed
  • Milestone changed from Awaiting Review to 6.6
  • Summary changed from The WordPress core password reset doesn't comply with new WCAG 2.2 success criterion to The WordPress core password reset needs to pre-populated the username to meet WCAG 2.2

Updating the ticket title to make it more clear what the needed change is for this.

Also removing needs design, as there's no visual change required; it just needs to populate the field, as far as I can tell.

#3 @joedolson
7 months ago

  • Owner set to joedolson
  • Status changed from new to accepted

#4 @joedolson
7 months ago

  • Summary changed from The WordPress core password reset needs to pre-populated the username to meet WCAG 2.2 to The WordPress core password reset needs to pre-populate the username to meet WCAG 2.2

#5 @sabernhardt
7 months ago

  • Description modified (diff)

mentioned in an Equalize Digital article

Last edited 7 months ago by sabernhardt (previous) (diff)

#6 @joedolson
7 months ago

Note: this should also apply to the link to log-in after installation.

This ticket was mentioned in PR #6332 on WordPress/wordpress-develop by rishavjeet.


6 months ago
#7

  • Keywords has-patch added; needs-patch removed

This PR mainly focuses upon the new feature of pre-populating the username field in the login form after password reset to meet WCAG 2.2 for accessibility.
For this feature, a new query parameter has been added to the password reset link that is sent to the email , so that after password reset the username field can be pre-field on the basis of the query parameter value user=
Screen-recording for demo

https://github.com/WordPress/wordpress-develop/assets/96969845/ed068467-12b0-4a3a-a042-aedc45dc5f77

Trac ticket: https://core.trac.wordpress.org/ticket/60726

Contributor name - Rishav Dutta

---

#8 @oglekler
3 months ago

I assume this should cover all possible cases, when there is a need to enter:

  1. Login after installation
  2. Login after restoring password
  3. Login after unsuccessful Login ✅
  4. Restoring password after unsuccessful Login if login is correct (we are providing this information, so, there is no secret that this user exists).

For me the empty field in the last case is really annoying, because you didn't manage to enter into the admin, most likely you tried several times, and now what you want to enter should be obvious. You can make a mistake in the username, but most likely you forgot the password.

What do you think?

#9 @joedolson
3 months ago

Yes, I'd agree that those are the cases that need to be covered. I assume the green check mark indicates that the current PR already meets that case?

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


3 months ago

#11 @rcreators
3 months ago

  • Keywords changes-requested added

Looks like there is still more work to do in this patch. I will create a new PR for this one to complete it this week. Else we will push it to 6.7.

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


3 months ago

#13 @sabernhardt
3 months ago

  • Milestone changed from 6.6 to 6.7

Moving the ticket to 6.7 for now.

This ticket was mentioned in PR #6928 on WordPress/wordpress-develop by @rishavdutta.


3 months ago
#14

This PR mainly focuses upon the new feature of pre-populating the username field in the login form after password reset to meet WCAG 2.2 for accessibility.
For this feature, a new query parameter has been added to the password reset link that is sent to the email , so that after password reset the username field can be pre-field on the basis of the query parameter value user=
Screen-recording for demo

https://github.com/WordPress/wordpress-develop/assets/96969845/ed068467-12b0-4a3a-a042-aedc45dc5f77

Trac ticket: https://core.trac.wordpress.org/ticket/60726

Contributor name - Rishav Dutta

---

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


2 months ago

@peterwilsoncc commented on PR #6928:


6 weeks ago
#16

This PR is duplicating a number of existing parameters that are passed around containing the user's login name. Unfortunately they use a variety of names: login, log and user_login. user_email is also used during registration.

Rather than create an additional parameter user, it would be good to reuse the existing parameters.

When a user resets their password, the link they are emailed contains both their username and the reset key. These are subsequently set to a session cookie and a redirect takes place to remove the data from the URL. This is to prevent a user from bookmarking data containing their username and reset code.

I think it would be beneficial to use a similar pattern for passing around the username in the improved flow removing the need to re-enter data. As part of the registration and password reset flows the goal is to prevent users from needing to reenter their username, however I don't think it wise to allow users to create a bookmark containing their username as it could lead to issues on shared computers.

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


5 weeks ago

Note: See TracTickets for help on using tickets.