[PR #13055] Add find item to tab menu #29373

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

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

State: closed
Merged: Yes


Summary of the Pull Request

Add Find to tab context menu as describe in issue #5633.

PR Checklist

Detailed Description of the Pull Request / Additional comments

Just wanted to solve Easy Starter issue, so any corrections/suggestions welcome. There's a couple of points I'm not sure of

  • Placement of item within menu, currently it's at the end before close tab block.
  • Should it be named longer, something like Find in Tab of just Find is fine?
  • The workaround for focus similar to tab rename is a bit annoying, especially because it required adding a method to TermControl but without it find window obviously opens without focus which is bad.

Validation Steps Performed

Open menu, press menu item try to find things via opened find dialog.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/13055 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Add `Find` to tab context menu as describe in issue #5633. ## PR Checklist * [x] Closes #5633 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA ## Detailed Description of the Pull Request / Additional comments Just wanted to solve `Easy Starter` issue, so any corrections/suggestions welcome. There's a couple of points I'm not sure of * Placement of item within menu, currently it's at the end before close tab block. * Should it be named longer, something like `Find in Tab` of just `Find` is fine? * The workaround for focus similar to tab rename is a bit annoying, especially because it required adding a method to `TermControl` but without it find window obviously opens without focus which is bad. ## Validation Steps Performed Open menu, press menu item try to find things via opened find dialog.
claunia added the pull-request label 2026-01-31 09:34:30 +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#29373