[PR #17885] Stop scrolling on output when search is open #31397

Closed
opened 2026-01-31 09:47:00 +00:00 by claunia · 0 comments
Owner

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

State: closed
Merged: Yes


  • Don't reset the position entirely when changing the needle
  • Don't change the scroll position when output arrives
  • Don't interfere with the search when output arrives constantly

Closes #17301

Validation Steps Performed

  • In pwsh, run 10000..20000 | % { sleep 0.25; $_ }
    • You can search for e.g. 1004 and it'll find 10 results.
    • You can scroll up and down past it and it won't snap back
      when new output arrives.
  • while ($true) { Write-Host -NoNewline "e[Ha"; sleep 0.0001; }`
    • You can cycle between the hits effortlessly. (This tests that
      the constantly reset OutputIdle event won't interfere.)
  • On input change, the focused result is near the previous one.
**Original Pull Request:** https://github.com/microsoft/terminal/pull/17885 **State:** closed **Merged:** Yes --- * Don't reset the position entirely when changing the needle * Don't change the scroll position when output arrives * Don't interfere with the search when output arrives constantly Closes #17301 ## Validation Steps Performed * In pwsh, run `10000..20000 | % { sleep 0.25; $_ }` * You can search for e.g. `1004` and it'll find 10 results. ✅ * You can scroll up and down past it and it won't snap back when new output arrives. ✅ * `while ($true) { Write-Host -NoNewline "`e[Ha"; sleep 0.0001; }` * You can cycle between the hits effortlessly. ✅ (This tests that the constantly reset `OutputIdle` event won't interfere.) * On input change, the focused result is near the previous one. ✅
claunia added the pull-request label 2026-01-31 09:47:00 +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#31397