[PR #13344] Dismiss Terminal-by-default banner on handoff #29491

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

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

State: closed
Merged: Yes


It's not useful to notify users that WT can be made the default if it's already
clearly being used for handoff. This commit will suppresses the banner then.

PR Checklist

Validation Steps Performed

  • Modify TerminalPage::ShowSetAsDefaultInfoBar to not check for
    CascadiaSettings::IsDefaultTerminalSet()
  • Set Terminal Dev as the default
  • Set incoming connections to open in the latest Terminal window
  • Delete state.json after every test below
  • Launching Terminal Dev shows the banner
    Launching cmd.exe dismisses the banner in the current Terminal
  • Launching cmd.exe launches Terminal Dev without banner
**Original Pull Request:** https://github.com/microsoft/terminal/pull/13344 **State:** closed **Merged:** Yes --- It's not useful to notify users that WT can be made the default if it's already clearly being used for handoff. This commit will suppresses the banner then. ## PR Checklist * [x] Closes #13314 * [x] I work here ## Validation Steps Performed * Modify `TerminalPage::ShowSetAsDefaultInfoBar` to not check for `CascadiaSettings::IsDefaultTerminalSet()` * Set Terminal Dev as the default * Set incoming connections to open in the latest Terminal window * Delete `state.json` after every test below * Launching Terminal Dev shows the banner ✅ Launching `cmd.exe` dismisses the banner in the current Terminal ✅ * Launching `cmd.exe` launches Terminal Dev without banner ✅
claunia added the pull-request label 2026-01-31 09:35:13 +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#29491