[PR #19212] Fix title being stuck after toggling "showTerminalTitleInTitlebar" #31749

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

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

State: closed
Merged: Yes


We already have the logic to fall back when the setting is disabled, we just also had a terrible eventing architecture and never used the Title getter that was smart enough to figure it out.

This pull request somewhat disentangles the mess that is title handling. For example, TerminalPage had a facility to get its current title! It was never used. Once we started using it, it wouldn't work for the Settings page... because it only read the title out of the active control.

Closes #12871
Closes #19139 (superseded)

**Original Pull Request:** https://github.com/microsoft/terminal/pull/19212 **State:** closed **Merged:** Yes --- We already have the logic to fall back when the setting is disabled, we just _also_ had a terrible eventing architecture and never used the `Title` getter that was smart enough to figure it out. This pull request somewhat disentangles the mess that is title handling. For example, TerminalPage had a facility to get its current title! It was never used. Once we started using it, it wouldn't work for the Settings page... because it only read the title out of the active control. Closes #12871 Closes #19139 (superseded)
claunia added the pull-request label 2026-01-31 09:49:18 +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#31749