[VT Render] Investigate not breaking lines for node-pty #1630

Closed
opened 2026-01-30 22:32:29 +00:00 by claunia · 2 comments
Owner

Originally created by @zadjii-msft on GitHub (Jun 12, 2019).

From MSFT:19012792


I didn't have any details in here. I presume that this was to track trying to not render a \r\n at the end of a line, so that a terminal would wrap the line itself. IIRC I tried that in RS5, but it broke something else, and there wasn't time to get it working for that release, so we reverted it.

Originally created by @zadjii-msft on GitHub (Jun 12, 2019). From MSFT:19012792 ----------- I didn't have any details in here. I presume that this was to track trying to not render a `\r\n` at the end of a line, so that a terminal would wrap the line itself. IIRC I tried that in RS5, but it broke something else, and there wasn't time to get it working for that release, so we reverted it.
claunia added the Area-RenderingIssue-BugNeeds-Tag-FixProduct-ConptyPriority-2 labels 2026-01-30 22:32:30 +00:00
Author
Owner

@ghost commented on GitHub (Jul 3, 2019):

Related: https://github.com/microsoft/terminal/issues/405

@ghost commented on GitHub (Jul 3, 2019): Related: https://github.com/microsoft/terminal/issues/405
Author
Owner

@zadjii-msft commented on GitHub (Jan 4, 2022):

I've been up and down the internal PRs around this time, and I can't find where this originally came from. The original bug has no details in it other than "it was a revert of something in the RS5 timeframe". I recall making some change to wrapping and having to revert it, but I can't find it, or the original bug.

Chances are, we've already fixed this along the way. I'm closing this.

@zadjii-msft commented on GitHub (Jan 4, 2022): I've been up and down the internal PRs around this time, and I can't find where this originally came from. The original bug has no details in it other than "it was a revert of something in the RS5 timeframe". I recall making some change to wrapping and having to revert it, but I can't find it, or the original bug. Chances are, we've already fixed this along the way. I'm closing this.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#1630