On Mon, Sep 12, 2011 at 11:11:09PM +0930, David Kettler wrote:
> If following a link during a hints interaction would also execute a
> script, the URL panel now includes the text "[script]" to warn the
> user.
This looks like a good feature. Can you resubmit against current HEAD?
Another style note, when an 'if' conditional spans more than one line, we
surround the body by curly braces, on their own lines.
if (elem.hasAttribute("onmousedown") ||
elem.hasAttribute("onclick"))
{
s.push("[script]");
}
This probably obviates the need for blank lines before and after the 'if',
but that's not a hard set rule.
Thanks
--
John Foerch
_______________________________________________
Conkeror mailing list
[email protected]
https://www.mozdev.org/mailman/listinfo/conkeror
