Skip to content
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

Command/Insert Modes #16

Merged
merged 6 commits into from
Nov 20, 2015
Merged

Command/Insert Modes #16

merged 6 commits into from
Nov 20, 2015

Conversation

jpoon
Copy link
Member

@jpoon jpoon commented Nov 19, 2015

Update: Ready to merge.

With this PR, implements basic key bindings in the various modes and ability to toggle between esc and insert keys

Supported Commands:

  • Insert Mode - i, I, a, A, o, O
  • Command Mode - h, j, k, l

The current mode is displayed at the bottom left status bar.

Somewhat addresses #13

* handles 'i' and 'I' to insert
* able to toggle between command (esc) and insert (w/ i or I)
@jpoon jpoon mentioned this pull request Nov 19, 2015
@guillermooo
Copy link
Member

One thing to keep in mind if that Vim uses key names like <CR>, <C-Home>, etc. We may want to translate to Vim-speak on input and back to a sensible output when needed.

@adriaanp
Copy link
Contributor

I would suggest that we register the commands as "vsc_vim.a" instead of "extension.vimMode_a"

What do you guys think?

Conflicts:
	test/scanner.test.ts
@jpoon
Copy link
Member Author

jpoon commented Nov 20, 2015

@adriaanp Absolutely. I'll do that.

Update: After taking a look at the other VSCode implemented commands, they keep things neat and tidy under namespaces (editor, etc). I think I'll keep with this convention but shorten it to extension.vim.a

@jpoon jpoon force-pushed the insert-mode branch 4 times, most recently from 4453c35 to d30450c Compare November 20, 2015 10:02
* Toggling between modes using 'esc' and insert keys

Supported Commands:
Insert Mode - i, I, a, A, o, O
Command Mode - h, j, k, l
jpoon added a commit that referenced this pull request Nov 20, 2015
@jpoon jpoon merged commit f53c37d into master Nov 20, 2015
@jpoon jpoon deleted the insert-mode branch November 20, 2015 10:21
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants