[PR #3280] [CLOSED] Don't reset the cursor X to 0 when we would otherwise wrap #25299

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/3280
Author: @DHowett-MSFT
Created: 10/22/2019
Status: Closed

Base: masterHead: dev/duhowett/gouge_a_huge_slash_in_the_screen_why_don't_ya


📝 Commits (1)

  • 59e1baf Don't reset the cursor X to 0 when we would otherwise wrap

📊 Changes

1 file changed (+0 additions, -5 deletions)

View changed files

📝 src/cascadia/TerminalCore/Terminal.cpp (+0 -5)

📄 Description

Fixes #3277.
Partially reverts #2965.

Summary of the Pull Request

Resetting the cursor position to X=0 made it act pretty strange when some applications wanted to do a full screen repaint.

PR Checklist

Validation Steps Performed

I spent two hours bisecting this, so I've hit Ctrl+B quite a lot in vim.


🔄 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/3280 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 10/22/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `dev/duhowett/gouge_a_huge_slash_in_the_screen_why_don't_ya` --- ### 📝 Commits (1) - [`59e1baf`](https://github.com/microsoft/terminal/commit/59e1baf821afb797bbea8c0d5c316b11cd5e36cb) Don't reset the cursor X to 0 when we would otherwise wrap ### 📊 Changes **1 file changed** (+0 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalCore/Terminal.cpp` (+0 -5) </details> ### 📄 Description Fixes #3277. Partially reverts #2965. ## Summary of the Pull Request Resetting the cursor position to X=0 made it act pretty strange when some applications wanted to do a full screen repaint. ## PR Checklist * [x] Closes #3277 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [x] Tests added/passed * [x] Requires documentation to be updated * [x] I've discussed this with core contributors already. ## Validation Steps Performed I spent two hours bisecting this, so I've hit <kbd>Ctrl+B</kbd> quite a lot in vim. --- <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:08:37 +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#25299