Multiple Selections Rendered with Search Box #7962

Closed
opened 2026-01-31 01:16:56 +00:00 by claunia · 4 comments
Owner

Originally created by @carlos-zamora on GitHub (May 5, 2020).

Originally assigned to: @carlos-zamora on GitHub.

Environment

Windows Terminal version (if applicable): 0.11.1251.0

Any other software?

Steps to reproduce

  1. (Optional) cd to our repo's src\cascadia\CascadiaPackage
  2. ls -R (or just print a lot of output)
  3. Open search box (ctrl+shift+f)
  4. Search png a few times (at least until you force the viewport to scroll)
  5. Scroll up about half a page (current selection should be past halfway down your viewport)
  6. Search png again (just hit enter)

Expected behavior

Only one selection is visible at a time (because only one exists).

Actual behavior

bug image

Originally created by @carlos-zamora on GitHub (May 5, 2020). Originally assigned to: @carlos-zamora on GitHub. # Environment ```none Windows Terminal version (if applicable): 0.11.1251.0 Any other software? ``` # Steps to reproduce 1. (Optional) cd to our repo's src\cascadia\CascadiaPackage 2. `ls -R` (or just print a lot of output) 3. Open search box (<kbd>ctrl+shift+f</kbd>) 4. Search `png` a few times (at least until you force the viewport to scroll) 5. Scroll up about half a page (current selection should be past halfway down your viewport) 6. Search `png` again (just hit enter) # Expected behavior Only one selection is visible at a time (because only one exists). # Actual behavior ![bug image](https://user-images.githubusercontent.com/11050425/81117065-5566bd80-8edb-11ea-8691-79fa09e3f563.png)
Author
Owner

@DHowett-MSFT commented on GitHub (May 5, 2020):

Search probably doesn't lock the renderer when changing its state or trigger scrolling properly. Want to look into this for 1.0?

@DHowett-MSFT commented on GitHub (May 5, 2020): Search probably doesn't lock the renderer when changing its state or trigger scrolling properly. Want to look into this for 1.0?
Author
Owner

@carlos-zamora commented on GitHub (May 5, 2020):

I can. I think Michael did a recent PR adding a bunch of those locks so I'll take a look at that and try it.

@carlos-zamora commented on GitHub (May 5, 2020): I can. I think Michael did a recent PR adding a bunch of those locks so I'll take a look at that and try it.
Author
Owner

@DHowett-MSFT commented on GitHub (May 7, 2020):

It is so assigned.

@DHowett-MSFT commented on GitHub (May 7, 2020): It is so assigned.
Author
Owner

@ghost commented on GitHub (May 13, 2020):

:tada:This issue was addressed in #5798, which has now been successfully released as Windows Terminal Release Candidate v0.11.1333.0 (1.0rc2).🎉

Handy links:

@ghost commented on GitHub (May 13, 2020): :tada:This issue was addressed in #5798, which has now been successfully released as `Windows Terminal Release Candidate v0.11.1333.0 (1.0rc2)`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v0.11.1333.0 (1.0rc2)) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#7962