| Summary: | Fix markup errors in self checkout and self checkin templates | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Owen Leonard <oleonard> |
| Component: | Self checkout | Assignee: | 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
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 |