| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| 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.  |