-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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 ctrl+c/ctrl+v to settings.json #5187
Comments
🎉This issue was addressed in #5217, which has now been successfully released as Handy links: |
@cinnamon-msft In Windows Terminal Version 1.7.1033.0 I don't have any keybindings in settings.json for copy and paste. Moreover, under Settings -> Actions, defaults are shown as copy being I might have had an existing settings json without keybindings before installing the Windows Terminal. From the bug reports here it is unclear to me what behavior is expected and what should the actual defaults for copy and paste be. I have the following questions:
|
@levicki Would you mind filing a new issue to track your problem? This issue's over a year old, and specifically was used to track adding ctrl+c/ctrl+v to the default |
@zadjii-msft I asked here intentionally because description of this fix does not reflect reality. It says:
My settings show different defaults when keybindings are empty. All I wanted is to be pointed to some place where current behavior is documented so I can determine whether I am having a problem or not before eventually submitting an issue. |
@levicki so, this is one of those annoying things we chose to do to preserve our userbase's sanity. We added the key bindings for people who first launch the terminal, or delete their settings, after 4/9/2020. It's part of the template settings file that is put on disk during startup if you don't already have settings. That template data is here- terminal/src/cascadia/TerminalSettingsModel/userDefaults.json Lines 60 to 64 in 8661dc5
If we had added them to the set of default key bindings in the application settings (the immutable ones), all the users who had installed and launched Terminal before 4/9/2020 would have suddenly lost ctrl+c/ctrl+v as verbatims. We place a reasonably high value on not breaking existing users¹, so this was the compromise we came to. ¹ we have done this before, and it tastes bad. Especially considering the input service discussion 😄 |
@DHowett Thanks for clarifying that part and linking to the template. Frankly, I was expecting from a Windows application that ctrl+c and ctrl+v are the built-in defaults for copy and paste even in the absence of keybindings in settings.json (of course with the ability to override them), but the graphical settings page is showing different defaults which also differ from the ones mentioned here. I sincerely doubt many Windows users know about and use those other key combos. |
Description of the new feature/enhancement
We should add
copy
andpaste
key bindings asctrl+c/v
in the settings.json file so people can see them and have the option to remove them.Additionally, there should be a comment in the file explaining that
ctrl+shift+c/v
are still shipped by default so they have a backup key binding if they choose to remove the above bindings.This would also close #3058
Proposed technical implementation details (optional)
The text was updated successfully, but these errors were encountered: