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

typo? use k instead of i to moveEditorToAboveGroup #1119

Merged
merged 5 commits into from
Jun 25, 2023

Conversation

ctstacey
Copy link
Collaborator

@ctstacey ctstacey commented Jan 2, 2023

No description provided.

@ctstacey ctstacey changed the title type? use k instead of i to moveEditorToAboveGroup typo? use k instead of i to moveEditorToAboveGroup Jan 2, 2023
@theol0403
Copy link
Member

Not a typo. Mentioned in readme. C-k clashes with many vscode bindings.

@justinmk justinmk closed this Jan 2, 2023
@ctstacey
Copy link
Collaborator Author

ctstacey commented Jan 3, 2023

Hi @theol0403,
Thanks for pointing it out in the readme, I missed that :)

I wonder though...
It looks like Ctrl+k is already being sent to neovim when not in insert mode (and the user has set neovim.ctrlKeysNormal causing ctrl key combos to be sent to neovim when not in insert mode).

vscode-neovim/package.json

Lines 791 to 796 in 30f268b

{
"command": "vscode-neovim.send",
"key": "ctrl+k",
"when": "editorTextFocus && neovim.init && neovim.mode != insert && neovim.ctrlKeysNormal",
"args": "<C-k>"
},

<C-w><C-k> mapping would only be used in normal and visual mode (n and x)

nnoremap <C-w><C-i> <Cmd>call VSCodeNotify('workbench.action.moveEditorToAboveGroup')<CR>
xnoremap <C-w><C-i> <Cmd>call VSCodeNotify('workbench.action.moveEditorToAboveGroup')<CR>

If user has not set neovim.ctrlKeysNormal, Ctrl+w wouldn't be sent in either normal or visual modes (n or x) regardless.

If you're willing to re-consider, please re-open this PR and I'll update the readme as well.
Sincerely,
Chris

@theol0403 theol0403 reopened this Jan 4, 2023
@ctstacey
Copy link
Collaborator Author

ctstacey commented Jan 4, 2023

Ready for review/suggestions, thanks :)

@ctstacey
Copy link
Collaborator Author

ctstacey commented Jan 4, 2023

Was the version already bumped?
Would this actually be released in 0.0.96?

@theol0403
Copy link
Member

This would be added to 0.0.97.

@theol0403
Copy link
Member

Thank you for all your PRs!

@theol0403 theol0403 merged commit 86ca759 into vscode-neovim:master Jun 25, 2023
theol0403 added a commit to theol0403/vscode-neovim that referenced this pull request Jun 27, 2023
chore(master): release 0.1.0

BEGIN_COMMIT_OVERRIDE
feat: Neovim toggle (vscode-neovim#1033)
fix: wrong VSCode selections if cursor is at start of selection vscode-neovim#1180
feat: add $NVIM_APPNAME option (vscode-neovim#1186)
feat: use k instead of i to moveEditorToAboveGroup (vscode-neovim#1119)
feat(ci): add automatic releases (and switch to semantic versioning)
END_COMMIT_OVERRIDE
theol0403 added a commit to theol0403/vscode-neovim that referenced this pull request Jun 27, 2023
BEGIN_COMMIT_OVERRIDE
feat: Neovim toggle (vscode-neovim#1033)
fix: wrong VSCode selections if cursor is at start of selection vscode-neovim#1180
feat: add $NVIM_APPNAME option (vscode-neovim#1186)
feat: use k instead of i to moveEditorToAboveGroup (vscode-neovim#1119)
feat(ci): add automatic releases (and switch to semantic versioning)
END_COMMIT_OVERRIDE
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.

None yet

3 participants