Project

General

Profile

Actions

Patch #21110

closed

Keep anchor (i.e. to a specific issue note) throughout login

Added by Jens Krämer over 8 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Category:
Accounts / authentication
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Problem

When not logged in and following a 'deep' link to a specific issue note, the user is first redirected to the sign in form, and after successful login the user is just sent to the top of the issue page because the anchor (i.e. #note-23) got lost along the way.

Solution

The anchor is not transferred to the server, so in order to keep it we need to introduce a bit of Javascript to preserve it when the login form is submitted.


Files

keep_anchor_on_signin.diff (1.06 KB) keep_anchor_on_signin.diff Jens Krämer, 2015-10-30 06:21
Actions #1

Updated by Jean-Philippe Lang over 8 years ago

Works fine with Firefox, but doesn't work with Chrome 46.

Actions #2

Updated by Jens Krämer over 8 years ago

Oh. Any JS errors? Chrome 45 works.

Actions #3

Updated by Jens Krämer over 8 years ago

I just checked in Chrome 47 / Linux and had a colleague check Chrome 46 / OSX 10.9 and both worked fine as well.

Actions #4

Updated by Jean-Philippe Lang over 8 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Target version set to 3.2.0

Sorry, my fault. Committed in r14767, thanks.

Actions #5

Updated by Christian Damen about 7 years ago

Workin' great. Thanks a lot Jean-Philippe :-)

Checked with Chrome 56, IE 11, Firefox 50.1

Actions

Also available in: Atom PDF