Bug 36003

Summary: Printing list from OPAC shows "Cookies" when CookieConsent enabled
Product: Koha Reporter: Mirjam Vantieghem <mirjam.vantieghem>
Component: ListsAssignee: Martin Renvoize <martin.renvoize>
Status: Needs documenting --- QA Contact: Kyle M Hall <kyle>
Severity: trivial    
Priority: P5 - low CC: david, fridolin.somers, kyle, m.de.rooy, martin.renvoize
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00
Bug Depends on: 35348    
Bug Blocks:    
Attachments: Bug 36003: Add cookie consent buttons to hidden list on print
Bug 36003: Add cookie consent buttons to hidden list on print
Bug 36003: Add cookie consent buttons to hidden list on print

Description Mirjam Vantieghem 2024-02-06 14:01:30 UTC
If system preference CookieConsent is set to "Required", the word "Cookies" appears at the bottem left on the last page, under the last result, when printing a list from the OPAC. Funny, but not desirable.
Not sure if similar to bug #12260.
Comment 1 Martin Renvoize 2024-02-07 13:38:16 UTC
Created attachment 161801 [details] [review]
Bug 36003: Add cookie consent buttons to hidden list on print

This patch adds #consentMoreInfo, #cookieConsentFooter and
.consentAcceptEssential to the hidden fields for OPAC printing.
Comment 2 David Nind 2024-02-10 03:07:51 UTC
Created attachment 162001 [details] [review]
Bug 36003: Add cookie consent buttons to hidden list on print

This patch adds #consentMoreInfo, #cookieConsentFooter and
.consentAcceptEssential to the hidden fields for OPAC printing.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2024-02-10 03:09:45 UTC
Testing notes (using koha-testing-docker):

1. Set the CookieConsent system preference to required.
2. Create a public list and add some items.
3. Go to the OPAC and view the list (don't log in, or click the 'Accept all cookies' prompt in the footer).
4. Print the list.
5. Note that "Accept all cookies" is printed near the bottom of the page.
6. Apply the patch.
7. Rebuild the CSS: yarn build
8. Restart everything: reset_all
9. Refresh the list page and print the list again - it should now not show "Accept all cookies" near the bottom of the page.
Comment 4 Marcel de Rooy 2024-03-01 10:48:32 UTC
Whole bunch of deps in PQA and SO too. Skipping for now.
Comment 5 Kyle M Hall 2024-05-03 13:38:05 UTC
Created attachment 166120 [details] [review]
Bug 36003: Add cookie consent buttons to hidden list on print

This patch adds #consentMoreInfo, #cookieConsentFooter and
.consentAcceptEssential to the hidden fields for OPAC printing.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 6 Katrin Fischer 2024-05-03 14:03:48 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 7 Fridolin Somers 2024-05-24 13:07:40 UTC
Depends on Bug 35348 not in 23.11.x