[Scenario] 2 Resize 2 Reflow #8006

Open
opened 2026-01-31 01:18:28 +00:00 by claunia · 1 comment
Owner

Originally created by @zadjii-msft on GitHub (May 7, 2020).

During 1.0, we discovered a ton of bugs in our resize with reflow implementation, and now that we need to keep both the conpty and the terminal in sync on these things, it's more important than ever to do these things right.

I'm collecting up all the straggling tasks I've found that might possibly be related.

This is a combo of "Make Reflow work correctly", and "Get rid of the spaghetti in the VtEngine related to line wrapping"

(see also #4200, the original reflow mega thread)

Issues

Things I think will all be fixed by #8000

PRs

Many of these are 1.0 PRs, but they've all got important context.

1.0 PRs
  • Don't reflow a line as wrapped if it broke on the last cell #5368
Originally created by @zadjii-msft on GitHub (May 7, 2020). During 1.0, we discovered a ton of bugs in our resize with reflow implementation, and now that we need to keep both the conpty and the terminal in sync on these things, it's more important than ever to do these things right. I'm collecting up all the straggling tasks I've found that might _possibly_ be related. This is a combo of "Make Reflow work correctly", and "Get rid of the spaghetti in the VtEngine related to line wrapping" (see also #4200, the original reflow mega thread) ## Issues * [ ] #796 Resize reflow tests * [ ] #797 Resize with Reflow loses leading spaces on resize operation * [ ] #3088 WSL Terminal Line Endings (the "exact wrap" bug) * #802 [host] If a line is exactly the width of the buffer and you increase the buffer width, it will wrap incorrectly resize reflow * [ ] #5228 Conpty: Additional `ScrollFrame` optimizations * [x] [REGRESSION >0.11.1251] Backward-kill-word after line wrapping in powershell doesn't clear exited line properly #5839 (PR #5870) * [ ] MSFT:19446208 - [This code in `TextBuffer::Reflow`](https://github.com/microsoft/terminal/blob/cf62922ad8da67464c517931b89f3fc098381b76/src/buffer/out/textBuffer.cpp#L2034-L2043) * [ ] #6566 Some text not displayed when the screen scrolls * [ ] #6901 When Copying, Line Wrapped Text Is Inconsistently Broken Into Multiple Lines - [ ] #10833 Multi-line paste with WSL omits and splices together lines - [ ] #11739 - [ ] #11809 * [ ] #7028 No line wrap at the bottom of the Terminal window when entering text in TCC * [ ] #7253 Un-snapping Terminal Placing Cursor in Middle of Previous Command's Output * (actually has a consistent repro) * [ ] #6987 Rendering errors in tmux split panes - [ ] This might also be "Tmux pane content gets garbled after refreshing tmux client #4029" * [ ] #7019 conpty exhibits pathological performance on scrolling region redraw (repaints entire screen) * [ ] #7479 * [ ] #7581 * [ ] #12440, which might just be #780 ### Things I think will all be fixed by #8000 * [ ] #6546 Command History on wrong line * [ ] #10130 * [ ] #10848 Cursor position is lost when resizing the window if not on the bottom line * [ ] #10868 also looks like this ## PRs Many of these are 1.0 PRs, but they've all got important context. <details> <summary> 1.0 PRs </summary> * [x] Don't remove spaces when printing a new bottom line with a background color #5550 * [x] Fix wrapped lines in `less` in Git for Windows #5771 * [x] Don't duplicate spaces from potentially-wrapped EOL-deferred lines #5398 * [x] Add a test to cover #5428 #5449 * [x] Fix copying wrapped lines by implementing better scrolling #5181 * [x] Emit lines wrapped due to spaces at the end correctly #5294 * [x] Fix an accidental regression from #5771 #5870 </details> * [ ] Don't reflow a line as wrapped if it broke on the last cell #5368
claunia added the Product-ConptyArea-CodeHealthIssue-Scenario labels 2026-01-31 01:18:29 +00:00
Author
Owner

@miniksa commented on GitHub (May 12, 2020):

2 Resize 2 Reflow

I'm pretty sure we're at least on Resize & Reflow: Tokyo Drift by now if not further into the series.

@miniksa commented on GitHub (May 12, 2020): > 2 Resize 2 Reflow I'm pretty sure we're at least on `Resize & Reflow: Tokyo Drift` by now if not further into the series.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#8006