[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #18 from Katrin Fischer --- (In reply to David Cook from comment #17) > (In reply to M from comment #16) > > Should I create a new bug about this? > > Yep. Create a new bug and have it depend on this one. Filed as: Bug 36337 - Hiding lists with OpacPublic breaks styling for language list -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Katrin Fischer changed: What|Removed |Added Blocks||36337 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36337 [Bug 36337] Hiding lists with OpacPublic breaks styling for language list -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #17 from David Cook --- (In reply to M from comment #16) > Should I create a new bug about this? Yep. Create a new bug and have it depend on this one. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244
M changed:
What|Removed |Added
CC||schodkowy.omegi-0r@icloud.c
||om
--- Comment #16 from M ---
Hello, this patch seems to have a side effect of the "Language" dropdown now
being awkwardly placed on center-right (regardless of whether user is logged in
our out). This is caused by the below CSS rules whose blame traces back to this
patch, and removing them fixes the issue.
#members {
+flex-grow: 1;
+ justify-content: flex-end;
Is this intentional change? I can only assume it's not, since I don't see any
mention of this in the discussion.
Screenshots of the problem: https://imgur.com/a/hDZvn7B
It looks like this patch conditionally hides cart+lists. The CSS above then
makes only language shown on the left, and the login on the right. But when
things aren't hidden, it changes the original position.
Should I create a new bug about this?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #15 from Lucas Gass --- Merge conflicts with 23.05.x, no backport. If needed please rebase. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Fridolin Somers changed: What|Removed |Added Status|Pushed to master|Pushed to stable CC||fridolin.som...@biblibre.co ||m Version(s)|24.05.00|24.05.00,23.11.01 released in|| --- Comment #14 from Fridolin Somers --- Pushed to 23.11.x for 23.11.01 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #13 from Katrin Fischer --- Pushed for 24.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Katrin Fischer changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||24.05.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #12 from Jonathan Druart --- Ideally we should redesign C4::Auth::get_template_and_user to not pass the lists in this case. But... well... you know... :D -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Jonathan Druart changed: What|Removed |Added Attachment #159188|0 |1 is obsolete|| --- Comment #11 from Jonathan Druart --- Created attachment 159225 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=159225&action=edit Bug 33244: Do not show lists in OPAC if OpacPublic is disabled This patch updates markup and CSS in the OPAC header so that the Cart and Lists controls don't appear when OpacPublic is disabled. The patch also moves the "Your cookies" link into the group of links with "Log in to your account" and "Search history." This makes the template logic a little simpler and also seems like a more logical grouping. To test, apply the patch and enable the CookieContent system preference. - View the OPAC with the OpacPublic system preference enabled. - If necessary, click the "Accept all cookies" button. - The "Your cookies" link should now appear in the header between the log-in link and the search history link. - Test the page at various browser widths to confirm that the header still responds well to changes. - Go back to system preferences in the staff client and change the OpacPublic system preference to "disabled." - Return to the OPAC and refresh the page. You should be redirected to the login page, and the Cart and Lists buttons should no longer appear in the header. - The links in the header for login and cookies should look correct at various browser widths. Signed-off-by: David Cook Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #10 from Jonathan Druart --- I could have guessed... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #9 from David Cook --- (In reply to David Cook from comment #8) > (In reply to Jonathan Druart from comment #7) > > "Log in to your account" is now on the left, is that expected? > > The test plan should've said that you need to run "perl build-resources.PL" > to re-generate the CSS. I should've noted it when testing due to its absence. My bad. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #8 from David Cook --- (In reply to Jonathan Druart from comment #7) > "Log in to your account" is now on the left, is that expected? The test plan should've said that you need to run "perl build-resources.PL" to re-generate the CSS. If you don't, it will appear on the left. But if you do run it, then "Log in to your account" is still on the right as expected. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Jonathan Druart changed: What|Removed |Added CC||jonathan.druart+koha@gmail. ||com --- Comment #7 from Jonathan Druart --- "Log in to your account" is now on the left, is that expected? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 David Cook changed: What|Removed |Added Attachment #159170|0 |1 is obsolete|| --- Comment #6 from David Cook --- Created attachment 159188 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=159188&action=edit Bug 33244: Do not show lists in OPAC if OpacPublic is disabled This patch updates markup and CSS in the OPAC header so that the Cart and Lists controls don't appear when OpacPublic is disabled. The patch also moves the "Your cookies" link into the group of links with "Log in to your account" and "Search history." This makes the template logic a little simpler and also seems like a more logical grouping. To test, apply the patch and enable the CookieContent system preference. - View the OPAC with the OpacPublic system preference enabled. - If necessary, click the "Accept all cookies" button. - The "Your cookies" link should now appear in the header between the log-in link and the search history link. - Test the page at various browser widths to confirm that the header still responds well to changes. - Go back to system preferences in the staff client and change the OpacPublic system preference to "disabled." - Return to the OPAC and refresh the page. You should be redirected to the login page, and the Cart and Lists buttons should no longer appear in the header. - The links in the header for login and cookies should look correct at various browser widths. Signed-off-by: David Cook -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 David Cook changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #5 from David Cook --- (In reply to David Cook from comment #4) > I'm not sure why these 2 unrelated changes are on the same bug report. Oh wait... I understand now. It is related because you're hiding the Cart and Lists. Yep yep yep. Gotcha. Makes sense. All good. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #4 from David Cook --- I'm not sure why these 2 unrelated changes are on the same bug report. I don't understand why only anonymous users see the "Your cookies" link, and why logged in users have to go into their account and look at "Consents". But I think that's an issue in master... so I'm not going to worry about it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 David Cook changed: What|Removed |Added CC||dc...@prosentient.com.au --- Comment #3 from David Cook --- *** Bug 30264 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Owen Leonard changed: What|Removed |Added Version|unspecified |master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #2 from Owen Leonard --- Created attachment 159170 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=159170&action=edit Bug 33244: Do not show lists in OPAC if OpacPublic is disabled This patch updates markup and CSS in the OPAC header so that the Cart and Lists controls don't appear when OpacPublic is disabled. The patch also moves the "Your cookies" link into the group of links with "Log in to your account" and "Search history." This makes the template logic a little simpler and also seems like a more logical grouping. To test, apply the patch and enable the CookieContent system preference. - View the OPAC with the OpacPublic system preference enabled. - If necessary, click the "Accept all cookies" button. - The "Your cookies" link should now appear in the header between the log-in link and the search history link. - Test the page at various browser widths to confirm that the header still responds well to changes. - Go back to system preferences in the staff client and change the OpacPublic system preference to "disabled." - Return to the OPAC and refresh the page. You should be redirected to the login page, and the Cart and Lists buttons should no longer appear in the header. - The links in the header for login and cookies should look correct at various browser widths. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 Owen Leonard changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33244] Do not show lists in OPAC if OpacPublic is disabled
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33244 --- Comment #1 from jpl --- Quick workaround I came up with. Works for me with Koha 22.11.10.000 Rosalie. This will only shows the lists when a user is logged in if opacpublic is set to false. --- /koha/src/masthead.inc 2023-10-25 23:02:01.538423660 +0200 +++ masthead.inc2023-10-25 23:03:45.558420570 +0200 @@ -37,6 +37,7 @@ [% END %] [% IF ( Koha.Preference( 'virtualshelves' ) == 1 ) %] + [% IF (( Koha.Preference( 'opacpublic' ) == 0 ) && ( logged_in_user )) || ( Koha.Preference( 'opacpublic' )) %] Lists @@ -81,6 +82,7 @@ [% END # / IF virtualshelves %] +[% END %] [% IF Koha.Preference( 'opacuserlogin' ) == 1 || Koha.Preference( 'EnableOpacSearchHistory') || Koha.Preference( 'opaclanguagesdisplay' ) %] -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
