Make WordPress Core

Opened 11 months ago

Last modified 5 months ago

#40605 assigned enhancement

Remove login form on checkemail=confirm screen

Reported by: rianrietveld Owned by: pratik028
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Login and Registration Keywords: good-first-bug has-patch early
Focuses: ui Cc:


When a user wants to retrieve her password on wp-login.php?action=lostpassword the next screen is wp-login.php?checkemail=confirm.

On that screen there is the message: Check your email for the confirmation link.
And below that is the login form again. This is confusing.

I found that users don't read the "check your email" message and fill out this form, as they think they are resetting their email with this form.

I suggest removing the login form from wp-login.php?checkemail=confirm and make the message "Check your email for the confirmation link." more stand out.

Attachments (4)

ticket#40605.patch (1.5 KB) - added by pratik028 10 months ago.
Added check-email switch case
40605.patch (1.5 KB) - added by bdbch 10 months ago.
40605.diff (1.3 KB) - added by hankthetank 7 months ago.
40605.2.patch (1.5 KB) - added by yashrs 5 months ago.
Added link(to login page) and text to ensure that user isn't confused where to go next.

Download all attachments as: .zip

Change History (14)

#1 @johnbillion
11 months ago

  • Keywords ux-feedback needs-patch good-first-bug added
  • Type changed from defect (bug) to enhancement
  • Version trunk deleted

+1 on this

10 months ago

Added check-email switch case

10 months ago

#3 @bdbch
10 months ago

Fixed the indentation on @pratik028 solution and fixed code style to the WP standard.

#4 @allisonplus
10 months ago

  • Keywords has-patch added; needs-patch removed

#5 @DrewAPicture
9 months ago

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

Assigning ownership to mark the good-first-bug as "claimed".

#6 @johnbillion
8 months ago

  • Keywords needs-patch added; ux-feedback has-patch removed

40605.patch needs more work. There are calls to log the user, and logic for handling a log out action, all of which aren't needed in this block.

7 months ago

#7 @hankthetank
7 months ago

This would be my first core contribution. I have removed the calls to log the user and the logic for handling a lot out action.

5 months ago

Added link(to login page) and text to ensure that user isn't confused where to go next.

#8 @yashrs
5 months ago

  • Keywords has-patch added; needs-patch removed

#10 @SergeyBiryukov
5 months ago

  • Keywords early added
  • Milestone changed from Awaiting Review to Future Release
Note: See TracTickets for help on using tickets.