Hi - great simple mod to add a nice feature.

Something that would be great is the addition of the destination argument in the "Log in" link so that the user's redirected to the page they were viewing when they decided to log in instead of staying on the /user page.

Using the comment login/register to reply functionality;
http://mysite.com/user/login?destination=comment/reply/5%2523comment_form

Or using the new alias created by this module;
http://mysite.com/login?destination=comment/reply/5%2523comment_form

Thanks for contributing this!

CommentFileSizeAuthor
#4 loginmenu_destination_99374.patch875 byteschristefano

Comments

rwohleb’s picture

rwohleb
he/him
Primary language English
commented
Assigned: Unassigned » rwohleb

Agreed. I'll put this on my todo list.

anders.fajerson’s picture

anders.fajerson commented

subscribing

christefano’s picture

christefano commented

Yes! I actually came to the issue queue to post this feature request.

christefano’s picture

christefano commented
Status: Active » Needs work
StatusFileSize
new loginmenu_destination_99374.patch875 bytes

I took a look at this again and thought I had it working, but for some reason it only works for logged in users. As you might imagine, that's the opposite of what we're trying to do.

Oh well, it's late. This needs work (I think I left an extra $destination variable in there) but it's most of the way there.