Bug 14579

Summary: 'You are not logged in' should link to log in page
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: Staff interfaceAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: chris, cnorthcott.work, gmcharlt, jonathan.druart, liz, tomascohen
Version: master   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14773
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14579: 'You are not logged in' should link to log in page
Bug 14579: 'You are not logged in' should link to log in page
Bug 14579: 'You are not logged in' should link to log in page
Bug 14579: 'You are not logged in' should link to log in page

Description Aleisha Amohia 2015-07-21 03:48:48 UTC
When you are not logged in and go to any page (for example an error page, cgi-bin/koha/errors/400.pl), it says 'You are not logged in |' in the top right in the style of a link, but does not redirect anywhere when you click it.
Comment 1 Aleisha Amohia 2015-07-21 03:51:53 UTC Comment hidden (obsolete)
Comment 2 cnorthcott.work 2015-08-14 03:49:33 UTC
Patch applies, link now works but alignment of Log In text and Help text are mismatched.
Comment 3 Jonathan Druart 2015-08-18 14:35:27 UTC
(In reply to cnorthcott.work from comment #2)
> Patch applies, link now works but alignment of Log In text and Help text are
> mismatched.

Aleisha, could you try to fix the alignment?
Comment 4 Joonas Kylmälä 2015-08-19 11:11:10 UTC Comment hidden (obsolete)
Comment 5 Aleisha Amohia 2015-08-27 03:59:06 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2015-08-27 07:24:12 UTC
Created attachment 42008 [details] [review]
Bug 14579: 'You are not logged in' should link to log in page

To test:
1) Go to any page on the staff client and make sure you are not logged in (ie an error page, cgi-bin/koha/errors/400.pl)
2) Try click on 'You are not logged in |'
3) Notice it is not really a link and doesn't take you anywhere
4) Apply patch
5) Refresh page
6) Click on what now says 'Log in |'
7) Confirm that you are redirected to the intranet log in page

Signed-off-by: Aleisha <aleishaamohia@hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Tomás Cohen Arazi 2015-08-27 16:28:12 UTC
Patch pushed to master.

Thanks Joonas, and congratulations for your first commit in Koha!
Comment 8 Liz Rea 2015-09-01 22:59:49 UTC
This patch moved the help menu from the top right, next to the login menu to inside the dropdown. I don't think we want to hide it like that.

Liz
Comment 9 Jonathan Druart 2015-09-02 07:56:54 UTC
Liz, it's the same place as when you are logged in. What do you suggest?
Comment 10 Liz Rea 2015-09-02 21:16:17 UTC
I'll make a different bug :)
Comment 11 Chris Cormack 2015-09-22 01:13:07 UTC
Pushed to 3.20.x will be in 3.20.4