Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Integrate CodeMirror v4 and implement multiple selections#7179

Merged
njx merged 146 commits intomasterfrom
cmv4
Mar 12, 2014
Merged

Integrate CodeMirror v4 and implement multiple selections#7179
njx merged 146 commits intomasterfrom
cmv4

Conversation

@njx
Copy link
Copy Markdown

@njx njx commented Mar 12, 2014

Landing the CodeMirror v4 work into master. See the multiple selections spec and the CodeMirror v4 upgrade guide for more details.

Thanks to @marijnh for all his work on multiple selections in CodeMirror v4.

Narciso Jaramillo and others added 30 commits November 14, 2013 12:57
[cmv4] Prevent modification of codemirror cursor position
Fix remaining cmv4 FindReplace unit tests
[cmv4] Fix LanguageManager unit test
Narciso Jaramillo and others added 24 commits March 6, 2014 13:23
[cmv4] Preserve multiple selections in various Esc cases
…dd a parameter to let the caller get all selections
[cmv4] Populate Quick Open from primary selection only
[cmv4] Fix merging of edits in inline color editor
[cmv4] Fix unit tests for getSelectedText() to reflect new behavior
…election is a range in the middle of the line to insert tab/spaces at beginning instead of replacing selection.
[cmv4] Implement Tab key heuristics for multiple selections
[cmv4] Fix case of hitting Tab at the beginning of a whitespace-only line
@njx
Copy link
Copy Markdown
Author

njx commented Mar 12, 2014

Smoke and unit tests all pass on Mac and Win. Merging.

njx pushed a commit that referenced this pull request Mar 12, 2014
Integrate CodeMirror v4 and implement multiple selections
@njx njx merged commit 098d360 into master Mar 12, 2014
@njx
Copy link
Copy Markdown
Author

njx commented Mar 12, 2014

Leaving the branch for now for reference (not deleting it). We can delete it later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants