Remove login form on checkemail=confirm screen

[rianrietveld]

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.

[johnbillion]

+1 on this

[SergeyBiryukov]

Related: #39638

[pratik028]

Added check-email switch case

[bdbch]

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

[DrewAPicture]

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

[johnbillion]

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]

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]

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

[SergeyBiryukov]

Related: #41514

[davidbaumwald]

Patch no longer applies cleaning, marking for a refresh.

[williampatton]

I refreshed the patch in this ticket and adjusted the link in the string to use sprintf so there is no dynamic content in the translation.

[johnbillion]

@williampatton Patch looks good. Can you upload some screenshots please?

[audrasjb]

Problem with the current patch

[audrasjb]

Hi,
There is a problem with the current patch. It displays two irrelevant errors.

[audrasjb]

There is also a JS error in the console, because of document.querySelector('form').classList.add('shake');. There is no form to "shake".

[bmartinent]

I just remove the wp_signon() call, how is useless here.

[audrasjb]

Thanks @bmartinent
This looks like a great workaround 👍
Just noting some edge cases, for example within user registration (see screenshot below): screen is empty, without any confirmation messsage.

[audrasjb]

edge case that occurs after user registration

[bmartinent]

This new patch do not interfere with the other checkemail cases. "wp-login.php?checkemail=newpass" and "wp-login.php?checkemail=registered"

[florianatwhodunit]

Before patch : form is displayed

[florianatwhodunit]

After patch : step 1

[florianatwhodunit]

After patch : step 2 : works fine

[florianatwhodunit]

Hi guys !

I've tested the last patch from @bmartinent .
It works well on my site. See screenshots above for reference ;)

thx for the patch, have a lovely day !
Florian

[audrasjb]

Patch looks good here as well, thanks.

[audrasjb]

Adding commit workflow keyword.

[SergeyBiryukov]

In 48303:

Login and Registration: Remove newpass value handling in wp-login.php.

This value is never set as of [15710].

See #40605.

[SergeyBiryukov]

In 48304:

Login and Registration: Hide the login form on the "check your email" step of new user registration or password reset.

This aims to reduce some confusion and make it clearer that the email should be checked before attempting to log in right away.

Props rianrietveld, pratik028, bdbch, johnbillion, hankthetank, yashrs, williampatton, audrasjb, bmartinent, florianatwhodunit, henry.wright, birgire, SergeyBiryukov.
Fixes #40605, #41514.

[SergeyBiryukov]

In 48305:

Login and Registration: Set a more appropriate page title for the "check your email" step of new user registration or password reset.

Follow-up to [48304].

See #40605, #41514.