The help text has "" and the special characters are not URL/HTML encoded, so this affects a page being valid XHTML since it thinks that is a tag and not plain text.

CommentFileSizeAuthor
#1 mibbit_irc-314725-1.module.patch1.79 KBChris Charlton

Comments

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented
Status: Active » Needs review
StatusFileSize
new mibbit_irc-314725-1.module.patch1.79 KB

Here's my patch for...

1. Fix the text to be [msg] instead, following the rest of the default help text. For those who have settings already saved, please edit the text yourself or reset to defaults in the administration form.

2. Fix XHTML output by changing border attribute of the iframe to frameborder, which is more cross-browser and is valid for XHTML sites.

3. Added name attribute of iframe, forgot this on last XHTML output patch. Name attribute needed for JavaScript interactivity and other DOM related calls.

kmillecam’s picture

kmillecam commented
Status: Needs review » Fixed

Committed to dev branch.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.