Bug 36909

Summary: Eliminate duplicate ID in cookie consent markup
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: Pushed to main --- QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.11.00
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!