Bug 40612

Summary: Eliminate duplicate element id in OPAC language menus
Product: Koha Reporter: Owen Leonard <oleonard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 40612: Eliminate duplicate element id in OPAC language menus

Description Owen Leonard 2025-08-07 15:51:01 UTC
If you have OpacLangSelectorMode set to "both" the page will have a duplicate id because the same menu include is used in the header and the footer.
Comment 1 Owen Leonard 2025-08-07 15:58:26 UTC
Created attachment 185229 [details] [review]
Bug 40612: Eliminate duplicate element id in OPAC language menus

This patch updates the include file used to display the OPAC language
menu in the header and footer. The include already makes use of a
"context" variable which tells it whether it's in the header or footer.
We can use the same logic to give unique ids for each instance.

To test, apply the patch and rebuild the OPAC CSS.

- Go to Administration -> System preferences -> OpacLangSelectorMode and
  set it to "both top and footer."
- Go to the OPAC and inspect the HTML for each language menu. The one in
  the header should have the id "header_langmenu." The one in the footer
  should have "footer_langmenu."

Sponsored-by: Athens County Public Libraries