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

feat: implements usage of insert to toggle between modes (as per #1787) #2356

Merged
merged 5 commits into from
Feb 26, 2018

Conversation

jpoon
Copy link
Member

@jpoon jpoon commented Feb 4, 2018

What this PR does / why we need it: Continuation of #2295. Haven't heard back from @gibranrosa, and so creating a new PR with his changes.

Which issue(s) this PR fixes. #1787

Special notes for your reviewer: Don't have an insert key on my keyboard to test.

@gibranrosa
Copy link

Hi! What conflicts? With the ?w=1 url parameter it could be checked that the only modifications was those which you made: https://github.com/VSCodeVim/Vim/pull/2295/files?w=1

@jpoon
Copy link
Member Author

jpoon commented Feb 9, 2018

I don't remember what the conflicts were. As mentioned in my CR comment, I shouldn't need to use that ?w=1 flag -- please just make the minimally necessary changes to the files. Changing whitespace makes it more difficult to track change history. If you'd like, please update your PR and we can close this one.

@jpoon jpoon changed the title @gibranrosa: partial implementation of insert key to toggle insert mode feat: implements usage of insert as per #1787 Feb 23, 2018
@jpoon jpoon changed the title feat: implements usage of insert as per #1787 feat: implements usage of insert to toggle between modes (as per #1787) Feb 23, 2018
@jpoon jpoon force-pushed the insert-key branch 5 times, most recently from f4c0e29 to e6171d4 Compare February 26, 2018 13:46
@VSCodeVim VSCodeVim deleted a comment from TravisBuddy Feb 26, 2018
@VSCodeVim VSCodeVim deleted a comment from TravisBuddy Feb 26, 2018
@jpoon jpoon merged commit d66a2db into master Feb 26, 2018
@jpoon jpoon deleted the insert-key branch February 26, 2018 14:44
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.

2 participants