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.
Created attachment 41106 [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
Patch applies, link now works but alignment of Log In text and Help text are mismatched.
(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?
Created attachment 41658 [details] [review] Bug 14579: 'You are not logged in' should link to log in page I fixed the aligment of the two links but the padding still needs to be set right. I also fixed mismatching html tags. 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
Created attachment 42004 [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>
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>
Patch pushed to master. Thanks Joonas, and congratulations for your first commit in Koha!
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
Liz, it's the same place as when you are logged in. What do you suggest?
I'll make a different bug :)
Pushed to 3.20.x will be in 3.20.4