| Summary: | OPAC detail and recent comments pages require corrections for XHTML validity | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Owen Leonard <oleonard> | 
| Component: | OPAC | Assignee: | Owen Leonard <oleonard> | 
| Status: | CLOSED FIXED | QA Contact: | Ian Walls <koha.sekjal> | 
| Severity: | minor | ||
| Priority: | P5 - low | CC: | chris, cnighswonger, jcamins, paul.poulain | 
| Version: | 3.4 | ||
| 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 fix
               Fix for Bug 7194 - OPAC detail and recent comments pages corrections for XHTML validity  | 
      ||
| 
 
        
          Description
        
        
          Owen Leonard
        
        
        
        
          2011-11-08 16:43:10 UTC
        
       
    Created attachment 6255 [details] [review] Proposed fix - Adding html filter to subtitle output to correct unescaped ampersands - Adding missing alt attributes to libravatar output - Removing unecessary markup from comments page Bonus: Adding html_break filter to output of comments to preserve line breaks Created attachment 6719 [details] [review] Fix for Bug 7194 - OPAC detail and recent comments pages corrections for XHTML validity - Adding html filter to subtitle output to correct unescaped ampersands - Adding missing alt attributes to libravatar output - Removing unecessary markup from comments page Bonus: Adding html_break filter to output of comments to preserve line breaks Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Adds alts, html escapement and a new html_break filter to various large text blobs. Also removes a yui-b div. Marking as Passed QA patch pushed, please test Patch does not apply cleanly to 3.4.x. Please rebase against 3.4.x and attach a new patch. I think it's not critical to fix this bug for 3.4 that is close to EOL. It's fixed in master though, marking RESO FIXED  |