[PR #15686] Let marks be cleared by clear (and friends) #30700

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

Original Pull Request: https://github.com/microsoft/terminal/pull/15686

State: closed
Merged: Yes


Summary of the Pull Request

Move scroll marks to TextBuffer, so they can be cleared by EraseInDisplay and EraseScrollback.

Also removes the namespacing on them.

References and Relevant Issues

Validation Steps Performed

  • cls works
  • Clear-Host works
  • clear works
  • the "Clear buffer" action works
  • They work when there's marks above the current viewport, and clear the scrollback
  • they work if you clear multiple "pages" of output, then scroll back to where marks previously were
  • resizing doesn't totally destroy the marks

PR Checklist

**Original Pull Request:** https://github.com/microsoft/terminal/pull/15686 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Move scroll marks to `TextBuffer`, so they can be cleared by EraseInDisplay and EraseScrollback. Also removes the namespacing on them. ## References and Relevant Issues * see also #11000 and #15057 * Resize/Reflow _doesn't_ work yet and I'm not attempting this here. ## Validation Steps Performed * `cls` works * `Clear-Host` works * `clear` works * the "Clear buffer" action works * They work when there's marks above the current viewport, and clear the scrollback * they work if you clear multiple "pages" of output, then scroll back to where marks previously were * resizing doesn't totally destroy the marks ## PR Checklist - [x] Closes #15426
claunia added the pull-request label 2026-01-31 09:42:26 +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#30700