[CLOSED] Hack yet another place where we need to use _codeMirror.replaceRange()...

[core-ai-bot]

Issue by njx
Friday May 03, 2013 at 17:31 GMT
Originally opened as adobe/brackets#3711

---

...in code hints to make them work in inline editors because of #1688. Sigh.

Fixes #3709.

To@eztierney for review. It looks like this is the single place where JS code hints actually edits the document, so I think this is the only place we have to patch this up. Let me know if there's another place we might have to do this fix.

---

njx included the following code: https://github.com/adobe/brackets/pull/3711/commits

[core-ai-bot]

Comment by njx
Friday May 03, 2013 at 17:34 GMT

---

/cc@dangoor, just FYI as something to watch out for in code hint providers.@adrocknaphobia -- we should really bring the "migrate to CodeMirror doc-linking" story (https://trello.com/c/cRAl0YHB) higher on the backlog to fix this issue once and for all...

[core-ai-bot]

Comment by eztierney
Friday May 03, 2013 at 17:59 GMT

---

yeah, I think this is the only place we do replaceRange.

Merging