Bug 41144

Summary: Fix markup errors in self checkout and self checkin templates
Product: Koha Reporter: Owen Leonard <oleonard>
Component: Self checkoutAssignee: Owen Leonard <oleonard>
Status: Needs Signoff --- QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Crowdfunding goal: 0
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 21266    
Bug Blocks: 41113    
Attachments: Bug 41144: Fix markup errors in self checkout and self checkin templates

Description Owen Leonard 2025-10-30 14:48:50 UTC
The W3C HTML validator complains of a number of minor issues in the self checkout and self checkin templates which should be addressed.
Comment 1 Owen Leonard 2025-10-30 15:43:52 UTC
Created attachment 188665 [details] [review]
Bug 41144: Fix markup errors in self checkout and self checkin templates

This patch fixes several markup errors in the self checkout and self
checkin templates, for example:

- Missing <fieldset>s (<legend> must be inside one)
- Stray ending </div>
- Missing <tr>

The patch also makes some changes to bring masthead-sci.inc and
masthead-sco.inc in sync, and makes a correction to a tab WRAPPER in
sco-main.tt.

To test, apply the patch and if necessary enable both self checkout
(WebBasedSelfCheck) and self checkin (SelfCheckInModule).

- Go to the self-checkout page and log in
- View source and copy the markup into the validator:
  https://validator.w3.org/#validate-by-input
- The only error you see should be one about a duplicate ID "langmenu"
  which is outside the scope of this patch.
- Perform the same test with the self checkin page.

Sponsored-by: Athens County Public Libraries