[PR #2755] [CLOSED] Bugfix: Line Selection Copy #25048

Open
opened 2026-01-31 09:06:56 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2755
Author: @carlos-zamora
Created: 9/14/2019
Status: Closed

Base: masterHead: dev/cazamor/bugfix-copy-newline


📝 Commits (1)

  • d7a8b9d Bugfix: line selection copy

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/buffer/out/textBuffer.cpp (+2 -2)

📄 Description

Summary of the Pull Request

Looks like this got moved from ConHost improperly. Thankfully, the comment helped track this down. I'll add tests on Monday.

References

PR Checklist

Validation Steps Performed

For CMD, Pwsh, and Ubuntu:

  • run dir/ls command
  • select first three lines
  • copy/copyTextWithoutNewlines
  • paste to notepad

Compare this behavior to ConHost.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/2755 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 9/14/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `dev/cazamor/bugfix-copy-newline` --- ### 📝 Commits (1) - [`d7a8b9d`](https://github.com/microsoft/terminal/commit/d7a8b9daa7a4c171824f5b0c13446dae2c4cc4ea) Bugfix: line selection copy ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/buffer/out/textBuffer.cpp` (+2 -2) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Looks like this got moved from ConHost improperly. Thankfully, the comment helped track this down. I'll add tests on Monday. <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes #2275 * [x] CLA signed * [ ] Tests added/passed * [x] Requires documentation to be updated * [x] I am a core contributor <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed For CMD, Pwsh, and Ubuntu: - run dir/ls command - select first three lines - copy/copyTextWithoutNewlines - paste to notepad Compare this behavior to ConHost. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:06:56 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#25048