ajaxorg/cloud9 — comment on commit b9a54547ace036953ecca483eedfe72a4abff385
File: client/ext/codetools/codetools.js line 27
By javruben:

This module needs a hook method. It should only create the AML when required by the codetool plugin. The way I see it, these colors are only really handy for css mode, so they should only apply to that mode. The aml should only be rendered when the user actually hovers over the text. So that should be separated from the rest of the code.

Alternatively I'd like to suggest to make this module pluggable. So to actually move the colorpicker in a separate plugin that hooks into this one. This plugin would then be the generic one that enables it's plugin based on different variables such as file extension.