Summary: | Missing <li> tag in circ-menu.inc | ||
---|---|---|---|
Product: | Koha | Reporter: | Garry Collum <gcollum> |
Component: | Templates | Assignee: | Garry Collum <gcollum> |
Status: | CLOSED FIXED | QA Contact: | Ian Walls <koha.sekjal> |
Severity: | minor | ||
Priority: | P5 - low | CC: | paul.poulain, wizzyrea |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Proposed patch
[Signed Off] Bug 7530: Missing <li> tag in circ-menu.inc |
Description
Garry Collum
2012-02-11 17:37:02 UTC
Created attachment 7576 [details] [review] Proposed patch Trivial patch that corrects an XHTML problem on several borrower pages. Can be tested by going to borrower checkout and validating the XHTML for the borrower information side menu. Created attachment 7645 [details] [review] [Signed Off] Bug 7530: Missing <li> tag in circ-menu.inc The address information of circ-menu.inc is not contained in <li> tags, this patch fixes it. Signed-off-by: Liz Rea <wizzyrea@gmail.com> verified new <li> in template. Looks good. Adds missing start element to list. This seems to have already been fixed in the .tt version of this file, so marking this simple template change as Passed QA. This patch add only a <li>, the </li> was already here, so the HTML is now more valid than before the patch This fix was included in the 3.6.x branch prior to 3.6.4. |