Forum:Code Review:AjaxRC 3

I've made some changes to AjaxRC, mainly style changes mixed in with some mw coding conventions. A breakdown is as follows:


 * Made whitespace more consistent with regards to parens.
 * Added a jshint directive per mw coding conventions and defined some non-standard globals.
 * Added the Wikia object to the function closure.
 * Changed tabs to 4 spaces for easier maintenance as tabs aren't standard characters on keyboards.
 * Only allow loading on viewing a page. The only instance I could think of when this might be useful otherwise was, but I didn't think it was especially needed. The main reason for this was to disable the script on editing as it undoes your changes every minute (or whatever the time is set to).
 * Removed the check for special namespace when using  within a conditional.   returns a string when in the special namespace and boolean  when outside of it.
 * Added a default for when the script is run on a page where none of the classes/ids normally used are found. This seems to only be an issue on pages with a user masthead from what I've seen.

It seems to be working well in my tests so far. I'd appreciate it if someone else could test it prior to it being pushed to the live version.