[PR #17777] Dismiss the snippet preview when there is no suggestions #31366

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

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

State: closed
Merged: Yes


Pretty obvious in retrospect. If there's no results, then we need to preview
nothing to make sure that we clear out any old previews.

Closes #17773

Additionally, while I was here:

I realized why it seems like the selected item is so wacky when you first open the sxnui:

  • on launch we're not scrolling to the bottom item (which makes it awkward in bottom up mode)
  • when we filter the list, we're maintaining the selection index, not the selection item.

Alas, that second part is... shockingly bodgy.

(you can review this as two separate commits for sanity's sake)

**Original Pull Request:** https://github.com/microsoft/terminal/pull/17777 **State:** closed **Merged:** Yes --- Pretty obvious in retrospect. If there's no results, then we need to preview _nothing_ to make sure that we clear out any old previews. Closes #17773 Additionally, while I was here: I realized why it seems like the selected item is so wacky when you first open the sxnui: * on launch we're not scrolling to the bottom item (which makes it awkward in bottom up mode) * when we filter the list, we're maintaining the selection _index_, not the selection _item_. Alas, that second part is... shockingly bodgy. (you can review this as two separate commits for sanity's sake)
claunia added the pull-request label 2026-01-31 09:46:48 +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#31366