[PR #3903] Re-add the keychords to the new tab flyout #25556

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

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

State: closed
Merged: Yes


Summary of the Pull Request

On this month's episode of "Mike accidentally breaks this":
Mike forgets that == is defined on a pair of winrt objects as "do these point to the SAME object", not "do they have the same values".

This just slipped right through code review.

References

Broken in #3825

PR Checklist

  • Closes Bug Report (#3896)
  • I work here
  • Tests added/passed
  • [n/a] Requires documentation to be updated

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Manually checked that these are still there

**Original Pull Request:** https://github.com/microsoft/terminal/pull/3903 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request On this month's episode of "Mike accidentally breaks this": Mike forgets that `==` is defined on a pair of winrt objects as "do these point to the _SAME_ object", not "do they have the same values". This just slipped right through code review. ## References Broken in #3825 ## PR Checklist * [x] Closes #3896 * [x] I work here * [ ] Tests added/passed * [n/a] Requires documentation to be updated ## Detailed Description of the Pull Request / Additional comments ## Validation Steps Performed Manually checked that these are still there
claunia added the pull-request label 2026-01-31 09:10:17 +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#25556