Bug 41144 - Fix markup errors in self checkout and self checkin templates
Summary: Fix markup errors in self checkout and self checkin templates
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Self checkout (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on: 21266
Blocks: 41113
  Show dependency treegraph
 
Reported: 2025-10-30 14:48 UTC by Owen Leonard
Modified: 2025-10-30 15:44 UTC (History)
0 users

See Also:
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:


Attachments
Bug 41144: Fix markup errors in self checkout and self checkin templates (90.31 KB, patch)
2025-10-30 15:43 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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