- unescaped ampersands - images without alt attributes
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