Bug 25390 - Login box is removed from opac-main.tt if Shibboleth or CAS is enabled
Summary: Login box is removed from opac-main.tt if Shibboleth or CAS is enabled
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-06 15:46 UTC by Lucas Gass
Modified: 2023-04-26 01:21 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass 2020-05-06 15:46:36 UTC
On the OPAC main page (opac-main.tt) if CAS or Shibboleth are enabled than we remove the login box on the right side of the page (#login). I am wondering if this is necessary:

1. It does not remove local login functionality as you can still login via the modal in the top right corner.
2. In multi library systems this can be a problem as some might use a SSO service while others would like to see the login box on the right side.
3. Maybe it would be simpler to keep the login box regardless of sign in method and hide it via CSS for libraries that don't want it?
Comment 1 Katrin Fischer 2020-05-06 21:46:33 UTC
I think the reasoning was that when you have a lot of different authentication methods activated they would clutter up the right column. Or the user would not find the Shibobleth/CAS options as they asssume they have to enter it there (where the other links don't show).

Hope that makes sense.
Comment 2 David Cook 2023-04-26 01:17:00 UTC
(In reply to Lucas Gass from comment #0)
> 3. Maybe it would be simpler to keep the login box regardless of sign in
> method and hide it via CSS for libraries that don't want it?

This is my inclination. Some libraries want both local login and SSO, while others don't. CSS is the easiest way to control this on a per-instance basis.

Bug 31064 was designed to make it easier to control the display of the local login.

That said, it looks like there are quite a few lines like "<p>If you do not have a Google account, but do have a local account, you can still log in: </p>" which are missing selectors at the moment...