A "request password" link should be added to the login form.

CommentFileSizeAuthor
#6 1414430-6.patch2.74 KBroderik
#1 Forgot_password_link-1414430-1.patch655 bytestommy kaneko

Comments

tommy kaneko’s picture

tommy kaneko commented
Assigned: tommy kaneko » Unassigned
Status: Active » Needs review
StatusFileSize
new Forgot_password_link-1414430-1.patch655 bytes

Very simple addition. The patch adds a "I forgot my password" as a link in the #description of the password field. The link creates a new window with the "request new password" page.

marcoka’s picture

marcoka commented
Status: Needs review » Reviewed & tested by the community

great, small one, works. i think target blank is good to not destory the current checkout progress.

luksak’s picture

luksak
he/him
commented

Works for me as well.

jjcarrion’s picture

jjcarrion
Primary language Spanish
Location Spain
commented

It works for me too, thanks!

sportel’s picture

sportel commented

This works nice! But... is it possible to send an e-mail with the D7 standard one-time-login-link immediately without opening a new screen, using the already entered e-mail address, and than show a message "An e-mail has been send ... bla bla bla"?

Thanks for your work,

Mike.

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 1414430-6.patch2.74 KB

We disagree a bit on what is the ideal approach to this.

The patch requires the user to fill the e-mail address a second time on the 'lost password' form. That's not ideal. My patch fixes this.

I did consider the suggestion in #5, but can't find a good UI solution for that. Since users don't expect a small link to perform an action, it would have to be a button, with big text -too big IMHO for a button.
Instead, I made the link open the separate password screen and return to the checkout page after sending the e-mail - using Drupal's 'destination' URL parameter.

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Related issues: +#2009986: Add capability to require login for existing and new users

This will probably need a reroll if #2009986: Add capability to require login for existing and new users lands.

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Parent issue: » #2580227: [META] 2.0 release
Related issues: -#2009986: Add capability to require login for existing and new users

New development will occur on the 2.x branch.

  • legolasbo committed bfe1d48 on 7.x-2.x 
    Issue #1414430 by legolasbo: Add "request password" link to the...
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.