Bug 36909

Summary: Eliminate duplicate ID in cookie consent markup
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: RESOLVED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: david, fridolin.somers, lucas
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes HTML validation warnings about duplicate IDs in the cookie consent markup for the OPAC and staff interface.
Version(s) released in:
24.11.00,24.05.02
Circulation function:
Bug Depends on: 27378    
Bug Blocks:    
Attachments: Bug 36909: Eliminate duplicate ID in cookie consent markup
Bug 36909: Eliminate duplicate ID in cookie consent markup

Description Owen Leonard 2024-05-21 14:46:41 UTC
The HTML validator complains about duplicate IDs in the cookie consent markup: One div#consentButtons in the header, one in the footer. These can be updated to be classes instead.
Comment 1 Owen Leonard 2024-05-21 14:54:48 UTC Comment hidden (obsolete)
Comment 2 David Nind 2024-05-23 05:42:18 UTC
Created attachment 167061 [details] [review]
Bug 36909: Eliminate duplicate ID in cookie consent markup

This patch changes markup and CSS in the OPAC and staff client to
eliminate HTML validator warnings about duplicate ids.

To test, apply the patch and rebuild all CSS (Bug 36909: Eliminate
duplicate ID in cookie consent markup).

- If necessary, set the "CookieConsent" system preference to "Require."
- Open the OPAC in a new private window (to prevent previous consents
  from hiding the consent messages).
- You should see a cookie consent bar across the bottom of the page. It
  should look correct, and its contents should reflow well at various
  browser widths.
- Click "Accept all cookies."
- In the header you should now see a "Your cookies" link.
- Click it and confirm that the contents of the modal look correct and
  reflow well at various browser widths.

- Perform the same tests in the staff interface.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Katrin Fischer 2024-06-27 13:33:00 UTC
Picked from QA queue for main.
Comment 4 Katrin Fischer 2024-06-27 15:26:45 UTC
Pushed for 24.11!

Well done everyone, thank you!
Comment 5 Lucas Gass (lukeg) 2024-07-19 15:36:57 UTC
Backported to 24.05.x for upcoming 24.05.02
Comment 6 Fridolin Somers 2024-07-22 09:57:29 UTC
Does not apply on 23.11.x
Maybe some other dependancy