-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Multi-Cursor Mode #587
Conversation
I can't believe I actually got it to work.
Conflicts: src/actions/actions.ts src/mode/modeHandler.ts src/register/register.ts src/util.ts
Conflicts: src/register/register.ts
Alright, this is done! I mean, there's still a bunch of work left to do, but it works reasonably well, and preserves all existing functionality, somehow, which is somewhat of a miracle. Anyways, @rebornix @jpoon @xconverge @sectioneight or anyone else who has time, feel free to give me feedback, though I realize that this PR is probably rather intimidating to review. The overview of what I did is, essentially:
Also:
|
This pr fixes #790 as well. |
We need to update |
Verified: This PR fixes code snippet issue mentioned in #479 (comment) |
Ah yeah I need to do updates for Travis. Also, I didn't even intend to fix #479. That's awesome. |
This looks great @johnfn. I can't say I grok every line of it, though. If there's a particular section you'd like some extra eyes on I'm happy to help. |
Moved to #811 and finished. |
Is this ever going to be revisited? It seems like it kind of got left half-finished.... |
The most requested feature in the history of VSCodeVim... Coming to a Visual Studio Code near you! 💥 🎆 💣 💀
Todo:
Stuff which should be fixed before release
Stuff which is not crucial for release but should be fixed at some point:
va(
works.)This is extra tricky. If you leave a mark
a
in multi cursor mode, we should probably store an array of all the positions that you left. Then if you are in normal mode, but then doma
, you would go into multi cursor mode at where all the a marks were left.This also makes me wonder if I could do something like
gma
(syntax pending) to add an additional location to the marks list. That seems really confusing for people other than me, however.