I think this a good place. We should not set is a property on the Editor object and if be define the function variables inside the initCodemirror function it is less concise/readable.