[PR #8979] Prevent context menu of tab renamer text box from canceling edit #27367

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

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

State: closed
Merged: Yes


PR Checklist

Detailed Description of the Pull Request / Additional comments

  • We dismiss the edit each time HeaderRenamerTextBox loses focus
  • Unfortunately, this applies also to scenario where the context menu
    (copy, paste, select, etc.) is open with the right-click
  • The fix is to ignore focus loss if HeaderRenamerTextBox().ContextFlyout() is open.
  • We can do it as upon the fly-out dismiss the text box regains the focus.

RenamerContextMenu

**Original Pull Request:** https://github.com/microsoft/terminal/pull/8979 **State:** closed **Merged:** Yes --- ## PR Checklist * [x] Closes https://github.com/microsoft/terminal/issues/8975 * [x] CLA signed. * [ ] Tests added/passed * [ ] Documentation updated. * [ ] Schema updated. * [ ] I've discussed this with core contributors already. ## Detailed Description of the Pull Request / Additional comments * We dismiss the edit each time `HeaderRenamerTextBox` loses focus * Unfortunately, this applies also to scenario where the context menu (copy, paste, select, etc.) is open with the right-click * The fix is to ignore focus loss if `HeaderRenamerTextBox().ContextFlyout()` is open. * We can do it as upon the fly-out dismiss the text box regains the focus. ![RenamerContextMenu](https://user-images.githubusercontent.com/4639110/106394866-90b10100-6407-11eb-8e92-627be4f70500.gif)
claunia added the pull-request label 2026-01-31 09:21:33 +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#27367