• The nl2br regex in the Html() formatter should have a config option
• There should be a RemoveJsFile method to complement the AddJsFile method.

Now, the RemoveJsFile method could return a bool of whether or not the file existed, but as it could be found multiple times I suppose maybe the number of times found would be a good return value. (E.g. the count of $keys)

Anyway, the changes can be seen in this commit here: http://github.com/LouisStAmour/Garden/commit/c1ba39dbb3ee53671f4bb229de6799fc901dd092

As coded, this plugin might or might not be compatible with Vanilla Replies or VanillaCommentScore as I encountered an error at one point and disabled both of them, though it might be the JS' fault.

Also, @Mark, is it possible to override JS or CSS from within a plugin, avoiding the need to remove 'discussion.js' just to add it again with a slightly different name? Or should we work on making the JS more extensible? The main incompatibility lies within jQuery's serialize() of the form, which doesn't seem to call the textarea's overridden val() to get the CKEditor's changes and so instead saves the original unmodified textarea that was hidden. One alternative could be a trigger for such changes before serializing the form.

Oh and another change I've forgotten to make is to ensure all IDs are unique for textareas. (Right now they totally aren't, so my start.js overcomes that as necessary.) And to swap to using Google for hosting jQuery, though that could be part of future performance optimizations like 1.2 has received, I guess.