[PR #10026] [MERGED] [WPF] Allows setting the WPF control background when setting the terminal theme #27859

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/10026
Author: @javierdlg
Created: 5/3/2021
Status: Merged
Merged: 5/4/2021
Merged by: @undefined

Base: mainHead: dev/jadelaga/ExternalBackgroundColor


📝 Commits (1)

  • 78d5c14 Allows setting the WPF control background when setting the terminal theme

📊 Changes

2 files changed (+8 additions, -3 deletions)

View changed files

📝 src/cascadia/WpfTerminalControl/TerminalControl.xaml.cs (+8 -2)
📝 src/cascadia/WpfTerminalControl/TerminalTheme.cs (+0 -1)

📄 Description

When syncing terminals across users (i.e. Liveshare shared terminals),
the terminal size is synced. This leads to having unused space around
the terminal which is the same color as the terminal's background
causing confusion as to what space is usable within the terminal.

Instead this change allows consumers to set the background color of the
control, separate from the terminal renderer's background, which makes
it easier to identify the edges of the terminal.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/10026 **Author:** [@javierdlg](https://github.com/javierdlg) **Created:** 5/3/2021 **Status:** ✅ Merged **Merged:** 5/4/2021 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/jadelaga/ExternalBackgroundColor` --- ### 📝 Commits (1) - [`78d5c14`](https://github.com/microsoft/terminal/commit/78d5c14872ac4bdc2bedc3d4f4722ad3f007342f) Allows setting the WPF control background when setting the terminal theme ### 📊 Changes **2 files changed** (+8 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/WpfTerminalControl/TerminalControl.xaml.cs` (+8 -2) 📝 `src/cascadia/WpfTerminalControl/TerminalTheme.cs` (+0 -1) </details> ### 📄 Description When syncing terminals across users (i.e. Liveshare shared terminals), the terminal size is synced. This leads to having unused space around the terminal which is the same color as the terminal's background causing confusion as to what space is usable within the terminal. Instead this change allows consumers to set the background color of the control, separate from the terminal renderer's background, which makes it easier to identify the edges of the terminal. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:24:44 +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#27859