This repository was archived by the owner on Sep 6, 2021. It is now read-only.
Repository navigation
This repository was archived by the owner on Sep 6, 2021. It is now read-only.
Expose CodeMirror.operation() and replaceRange() on Document #803
Description
Currently, it's hard to write a command that does text edits without talking directly to Editor._codeMirror, and we don't really want that. If we simply exposed operation() and replaceRange(), it would go a long way toward fixing this.
I'm fudging a little by marking this "code cleanup," but I think it would be a very simple change -- little more than proxying down to the matching methods on Document._masterEditor.
Activity
peterflynn commentedon May 3, 2012
Nominating for sprint 8 since it ties in nicely with our extensibility goals
njx commentedon May 8, 2012
Assigning to Peter, but let's talk about whether we want to expose the APIs or the object itself.
pthiess commentedon May 11, 2012
moved to Sprint 9
peterflynn commentedon May 24, 2012
Architectural discussion of this work is ongoing here: https://groups.google.com/forum/?fromgroups#!topic/brackets-dev/ok20uf8Q_a4
peterflynn commentedon Jun 1, 2012
After discussion with NJ & Glenn, we decided to move this out to the beginning of Sprint 10.
peterflynn commentedon Jun 14, 2012
Merged by pull request #980. Document now has equivalents to CodeMirror.operation(), replaceRange(), getRange(), and getLine(). (And Document.getText()/setText() have better semantics).