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 | ||
Change sponsored?: | --- | 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 |