-
Notifications
You must be signed in to change notification settings - Fork 28.9k
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
Copy from terminal should not retain new line characters for wrapped lines #23142
Comments
I can also reproduce this behavior in F#, with the Ionide F# plugin. Looks like copying a wrapped line from the terminal (that is, a long line that was visually wrapped) ends up copying real LF or CR/LF characters from the terminal, when they were not present in the original line. Related: if I print a long line to the terminal, then resize VS Code, the line does NOT re-wrap, because the terminal has already inserted linefeeds into the text it printed. This results in remarkably ugly terminal output: either some part of the long line is no longer visible, or there are huge sections of whitespace when the line is wrapped too early. Perhaps that should be tracked as a separate (but related) bug? |
Upstream issue: xtermjs/xterm.js#443 Related: xtermjs/xterm.js#609 |
Merged xtermjs/xterm.js#693 into release branch, fix should come in tomorrow's insiders. |
Verified to early and discovered that under Windows the newline is \n which doesn't let you paste into notepad. Opened #29856 |
Fixation 8076a19
Date 2017-03-08T14:00:46.854Z
Shell 1.4.6
Handler 53.0.2785.143
Node 6.5.0:
Steps to Reproduce:
Words "source_port" and "flag_push" and quotes on ' "checksum" ' are changed. So when I copy this text to https://jsonformatter.curiousconcept.com/ I've got invalid json.
What words are broken, depends on VSCode window size. When I made it large or small, different words became corrupted.
The text was updated successfully, but these errors were encountered: