[PR #8588] Introduce search status #27245

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

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

State: closed
Merged: No


Summary of the Pull Request

This is a PoC for:

  • Search status in SearchBox (aka number of matches + index of the current match)
  • Live search (aka search upon typing)
  • Highlighting all matches (probably should try later on separately)

PR Checklist

Detailed Description of the Pull Request / Additional comments

  • Introduced this optionally (global setting to enable it)
  • The approach is following:
    • Every time the filter changes, enumerate all matches
    • Upon navigation just take the relevant match and select it

Validation Steps Performed

  • Manual testing, more tests required - this is a PoC only
**Original Pull Request:** https://github.com/microsoft/terminal/pull/8588 **State:** closed **Merged:** No --- ## Summary of the Pull Request This is a PoC for: * Search status in SearchBox (aka number of matches + index of the current match) * Live search (aka search upon typing) * ~Highlighting all matches~ (probably should try later on separately) ## PR Checklist * [x] Closes https://github.com/microsoft/terminal/issues/6319 * [x] CLA signed. * [ ] Tests added/passed * [ ] Documentation updated. * [ ] Schema updated * [ ] I've discussed this with core contributors already. <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments * Introduced this optionally (global setting to enable it) * The approach is following: * Every time the filter changes, enumerate all matches * Upon navigation just take the relevant match and select it ## Validation Steps Performed * Manual testing, more tests required - this is a PoC only
claunia added the pull-request label 2026-01-31 09:20:51 +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#27245