[PR #14708] Update the titlebar visibility when we gain/lose focus too #30221

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

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

State: closed
Merged: Yes


We forgot to updateTheme again when we change the titlebar color. That would result in us setting the titlebar visibility only when the settings were reloaded, but if the unfocused BG was opaque, and the focused was transparent, we'd use the current unfocused color to set the titlebar visibility.

Also, this fixes a bug with tabRow.BG: terminalBackground

from teams for brevity

tabRow.BG = terminalBackground might not work, and here's why

the termcontrol's BG brush might be (r,g,b, 255), with an Opacity of 0

so my "use mica when the brush's A is <1.0" doesn't work

closes #14563

**Original Pull Request:** https://github.com/microsoft/terminal/pull/14708 **State:** closed **Merged:** Yes --- We forgot to updateTheme again when we change the titlebar color. That would result in us setting the titlebar visibility only when the settings were reloaded, but if the unfocused BG was opaque, and the focused was transparent, we'd use the current _unfocused_ color to set the titlebar visibility. Also, this fixes a bug with `tabRow.BG: terminalBackground` from teams for brevity > tabRow.BG = terminalBackground might not work, and here's why > > the termcontrol's BG brush might be (r,g,b, 255), with an Opacity of 0 > > so my "use mica when the brush's A is <1.0" doesn't work > closes #14563
claunia added the pull-request label 2026-01-31 09:39:24 +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#30221