Implement copy and paste for keyboard shortcuts#1072
Closed
d-bingham wants to merge 4 commits intomicrosoft:masterfrom
d-bingham:master
Closed
Implement copy and paste for keyboard shortcuts#1072d-bingham wants to merge 4 commits intomicrosoft:masterfrom d-bingham:master
d-bingham wants to merge 4 commits intomicrosoft:masterfrom
d-bingham:master
Conversation
…dd the option to strip line feeds from pasted text.
dlong11
reviewed
May 31, 2019
…nd renamed it appropriately.
zadjii-msft
requested changes
May 31, 2019
Member
zadjii-msft
left a comment
There was a problem hiding this comment.
I'd rather we break this into two PR's:
- one for adding the copy and paste keybindings (which will probably require minimal changes, if any to get in)
- another for the line endings conversion on paste.
The first is a much simpler change that I think we'd all easily get behind, with the second might involve a bit more discussion.
Contributor
Author
|
Closing this to split into two PRs. |
This was referenced Jun 1, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of the Pull Request
Attaches copy/paste functionality to their keyboard shortcuts and adds an option to improve interaction between "Windows-space" clipboards and "Linux-space" Terminals.
References
#968
PR Checklist
Detailed Description of the Pull Request / Additional comments
This connects the (mostly) pre-existing copy/paste mechanics to the keyboard shortcut mechanics, and adds a new option which makes multi-line pasting in WSL actually usable.
The new option, convertPasteLineEndings does what it says -- replacing Windows-space CRLF pairs with Unix-space LFs in text pasted to the console. This option applies to both the keyboard-shortcut paste and the right-click paste. Without this most multiline text pasted into Terminal will be "double-spaced" due to the Windows-style CRLF pairs. Furthermore, any multiline text copied from Terminal (in trim whitespace mode) generates CRLF pairs, which guarantees double-spacing when copying from a WSL Terminal session.
Changes:
Other notes: