-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Move default comment toggle binding to pound #3799
base: master
Are you sure you want to change the base?
Move default comment toggle binding to pound #3799
Conversation
This is a common rebind in both the helix and kakoune community. Some reasons this is desirable: * Pound is a common symbol for commenting in many languages * Pound is not bound to anything else, and it is not obvious what else it would be bound to * `C-c` should really be reserved for cancelling subshells from `:sh`, `|` and friends Most sensible alternatives are taken: * `/` is search * `*` adds selection to search register * `-` is not common, and though not currently taken, may make sense as the opposite of `_`, for another PR to implement. * `c` change selection * `A-c` change selection w/o yank * `C` copy selection down * `A-C` copy selection up
C-c does work for closing completion menus / signature help so I think it should work to have it cancel long-running subshell commands if there is one running. I'm split on this. I think it would be really intuitive for languages with |
What's the appropriate way to move this forward? |
I've been testing this out but I'd like to see more opinions. |
I think it is a far better choice to use
|
Why not |
I think |
I think this is a good choice since For context:
So I guess for non-modal editors or vim-based keybindings ( If we want to make it easy for anyone (including from the ones mentioned above), Given that this is type of comment is also quite widely used, I created a pull request to set it as default instead of But for helix and kakoune case, Once the docs are updated, I guess this can be merged given that there is no one opposing here. |
C-/ is not nice for keyboards with umlaut-keys -> have to press 3 keys (ctrl+shift+7) |
This is a common rebind in both the helix and kakoune community.
Some reasons this is desirable:
it would be bound to
C-c
should really be reserved for cancelling subshells from:sh
,|
and friendsMost sensible alternatives are taken:
/
is search*
adds selection to search register-
is not common, and though not currently taken, may make senseas the opposite of
_
, for another PR to implement.c
change selectionA-c
change selection w/o yankC
copy selection downA-C
copy selection up