-
-
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
Add default-yank-register option #11430
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The use-case in #11398 can be covered by rebinding commands, see #2038 (comment)
While this is a small and straightforward patch, I'm not inclined to approve it unless there's another use-case that isn't already possible by rebinding commands
Rust is hard 😔 |
See #2038 (comment), this actually might be preferrable to remapping commands since the clipboard yanking commands don't accept a different register. (So if you remap your keys to use those then you can't yank to another register) |
Co-authored-by: Michael Davis <[email protected]>
Co-authored-by: Michael Davis <[email protected]>
oops Co-authored-by: Michael Davis <[email protected]>
I gotta figure out the system clipboard integration in Kitty. Might as well wait for helix-editor/helix#11430, as changing the bindings breaks usage of other registries. Signed-off-by: William Artero <[email protected]>
Co-authored-by: Michael Davis <[email protected]>
Co-authored-by: Michael Davis <[email protected]>
Co-authored-by: Michael Davis <[email protected]>
For this field you want to supply the register character alone, so just [editor]
default-yank-register = "+" should do the trick. The "characterin" part is probably because we ignore newlines in the statusline and that is probably a split in a line in the error message |
Solution for the #11398