WordPress.org

Make WordPress Core

Opened 9 months ago

Last modified 3 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:

Description

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 8 months ago.
Added check-email switch case
40605.patch (1.5 KB) - added by bdbch 8 months ago.
40605.diff (1.3 KB) - added by hankthetank 5 months ago.
40605.2.patch (1.5 KB) - added by yashrs 3 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
9 months ago

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

+1 on this

@pratik028
8 months ago

Added check-email switch case

@bdbch
8 months ago

#3 @bdbch
8 months ago

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

#4 @allisonplus
8 months ago

  • Keywords has-patch added; needs-patch removed

#5 @DrewAPicture
7 months ago

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

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

#6 @johnbillion
6 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.

@hankthetank
5 months ago

#7 @hankthetank
5 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.

@yashrs
3 months ago

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

#8 @yashrs
3 months ago

  • Keywords has-patch added; needs-patch removed

#10 @SergeyBiryukov
3 months ago

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