[PR #14456] Fix a couple issues with experimental.useBackgroundImageForWindow #30101

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

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

State: closed
Merged: Yes


This fixes two issues with experimental.useBackgroundImageForWindow I discovered while looking at #14260

  • It looks like the opacity of the whole-window BG image wouldn't hot reload if the path didn't.
  • set useBGForWindow:true, focus a pane with an image, then set it to useBGForWindow:false, and observe a pane with <100 opacity. You'll be able to see the BG image left behind!

These are pretty easy to miss, so I can see how it happened.

I don't think this technically closes that thread, though. Ultimately, I think OP's settings were just wrong (and possible didn't hot-reload). There's another, trickier bit I'm discussing in that thread, that might deserve its own separate follow-up for discussion.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/14456 **State:** closed **Merged:** Yes --- This fixes two issues with `experimental.useBackgroundImageForWindow` I discovered while looking at #14260 * It looks like the opacity of the whole-window BG image wouldn't hot reload if the path didn't. * > set useBGForWindow:true, focus a pane with an image, then set it to useBGForWindow:false, and observe a pane with <100 opacity. You'll be able to see the BG image left behind! These are pretty easy to miss, so I can see how it happened. I don't think this _technically_ closes that thread, though. Ultimately, I think OP's settings were just wrong (and possible didn't hot-reload). There's another, trickier bit I'm discussing in that thread, that might deserve its own separate follow-up for discussion.
claunia added the pull-request label 2026-01-31 09:38:39 +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#30101