Skip to content

Commit f8202e2

Browse files
TechypandaTechypanda
committed
Setting to set pane highlight active, inactive and broadcast color
This commit adds a new setting in which you can configure the active highlight color of panes aswell as inactive and broadcast.
1 parent e332c67 commit f8202e2

File tree

7 files changed

+131
-51
lines changed

7 files changed

+131
-51
lines changed

‎doc/cascadia/profiles.schema.json‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2000,6 +2000,24 @@
20002000
}
20012001
}
20022002
},
2003+
"PaneTheme": {
2004+
"additionalProperties": false,
2005+
"description": "A set of properties for customizing the appearance of the panes",
2006+
"properties": {
2007+
"activeBorderColor": {
2008+
"description": "The color of the pane border when the pane is active",
2009+
"$ref": "#/$defs/ThemeColor"
2010+
},
2011+
"inactiveBorderColor": {
2012+
"description": "The color of the pane border when the pane is inactive",
2013+
"$ref": "#/$defs/ThemeColor"
2014+
},
2015+
"broadcastBorderColor": {
2016+
"description": "The color of the pane border when broadcasted",
2017+
"$ref": "#/$defs/ThemeColor"
2018+
}
2019+
}
2020+
},
20032021
"WindowTheme": {
20042022
"additionalProperties": false,
20052023
"description": "A set of properties for customizing the appearance of the window itself",
@@ -2056,6 +2074,9 @@
20562074
},
20572075
"window": {
20582076
"$ref": "#/$defs/WindowTheme"
2077+
},
2078+
"pane": {
2079+
"$ref": "#/$defs/PaneTheme"
20592080
}
20602081
}
20612082
},

‎src/cascadia/TerminalApp/TerminalPage.cpp‎

Lines changed: 75 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -4689,11 +4689,10 @@ namespace winrt::TerminalApp::implementation
46894689
}
46904690

46914691
const auto theme = _settings.GlobalSettings().CurrentTheme();
4692-
auto requestedTheme{ theme.RequestedTheme() };
4692+
const auto requestedTheme{ theme.RequestedTheme() };
46934693

46944694
{
4695-
_updatePaneResources(requestedTheme);
4696-
4695+
_updatePaneResources(requestedTheme, theme.Pane());
46974696
for (const auto& tab : _tabs)
46984697
{
46994698
if (auto terminalTab{ _GetTerminalTabImpl(tab) })
@@ -4796,67 +4795,94 @@ namespace winrt::TerminalApp::implementation
47964795
}
47974796
}
47984797

4798+
4799+
Color TerminalPage::_colorFromKey(const ResourceDictionary& resourceDictionary, const ElementTheme& requestedTheme, const IInspectable& colorKey)
4800+
{
4801+
const auto defaultColor = Colors::Black();
4802+
if (!resourceDictionary.HasKey(colorKey))
4803+
{
4804+
return defaultColor;
4805+
}
4806+
return winrt::unbox_value_or<Color>(
4807+
ThemeLookup(resourceDictionary, requestedTheme, colorKey),
4808+
defaultColor
4809+
);
4810+
}
4811+
4812+
Color TerminalPage::_parseThemeColorToColor(
4813+
const ThemeColor& colorToCopy,
4814+
const ResourceDictionary& resourceDictionary,
4815+
const ElementTheme& requestedTheme,
4816+
const IInspectable& colorKey
4817+
)
4818+
{
4819+
switch (colorToCopy.ColorType())
4820+
{
4821+
case ThemeColorType::Accent:
4822+
return _colorFromKey(resourceDictionary, requestedTheme, colorKey);
4823+
case ThemeColorType::Color:
4824+
const auto rawColor = colorToCopy.Color();
4825+
return Color{
4826+
rawColor.A,
4827+
rawColor.R,
4828+
rawColor.G,
4829+
rawColor.B
4830+
};
4831+
case ThemeColorType::TerminalBackground:
4832+
// does not work
4833+
const auto terminalBg = ThemeColor::FromTerminalBackground().Color();
4834+
return Color{
4835+
terminalBg.A,
4836+
terminalBg.R,
4837+
terminalBg.G,
4838+
terminalBg.B
4839+
};
4840+
default:
4841+
assert(false && "unknown type for color type in theme color"); // should never be reached
4842+
return winrt::Windows::UI::Color{};
4843+
}
4844+
}
4845+
47994846
// Function Description:
48004847
// - Attempts to load some XAML resources that Panes will need. This includes:
48014848
// * The Color they'll use for active Panes's borders - SystemAccentColor
48024849
// * The Brush they'll use for inactive Panes - TabViewBackground (to match the
48034850
// color of the titlebar)
48044851
// Arguments:
48054852
// - requestedTheme: this should be the currently active Theme for the app
4853+
// - activeBorderColor: the pane's border color for the application when it is active
4854+
// - inactiveBorderColor: the pane's border color for the application when it is inactive
4855+
// - broadcastBorderColor: the pane's border color for the application when it is broadcast
48064856
// Return Value:
48074857
// - <none>
4808-
void TerminalPage::_updatePaneResources(const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme)
4858+
void TerminalPage::_updatePaneResources(const ElementTheme& requestedTheme, const PaneTheme& paneTheme)
48094859
{
4860+
const auto paneActiveBorderColor = paneTheme ? paneTheme.ActiveBorderColor() : nullptr;
4861+
const auto paneInactiveBorderColor = paneTheme ? paneTheme.InactiveBorderColor() : nullptr;
4862+
const auto broadcastBorderColor = paneTheme ? paneTheme.BroadcastBorderColor() : nullptr;
48104863
const auto res = Application::Current().Resources();
4811-
const auto accentColorKey = winrt::box_value(L"SystemAccentColor");
4812-
if (res.HasKey(accentColorKey))
4864+
const auto tmp = ThemeColor::FromTerminalBackground();
4865+
const auto color = tmp.Color();
4866+
if (tmp)
48134867
{
4814-
const auto colorFromResources = ThemeLookup(res, requestedTheme, accentColorKey);
4815-
// If SystemAccentColor is _not_ a Color for some reason, use
4816-
// Transparent as the color, so we don't do this process again on
4817-
// the next pane (by leaving s_focusedBorderBrush nullptr)
4818-
auto actualColor = winrt::unbox_value_or<Color>(colorFromResources, Colors::Black());
4819-
_paneResources.focusedBorderBrush = SolidColorBrush(actualColor);
48204868
}
4821-
else
4869+
if (color.A)
48224870
{
4823-
// DON'T use Transparent here - if it's "Transparent", then it won't
4824-
// be able to hittest for clicks, and then clicking on the border
4825-
// will eat focus.
4826-
_paneResources.focusedBorderBrush = SolidColorBrush{ Colors::Black() };
4827-
}
4828-
4829-
const auto unfocusedBorderBrushKey = winrt::box_value(L"UnfocusedBorderBrush");
4830-
if (res.HasKey(unfocusedBorderBrushKey))
4831-
{
4832-
// MAKE SURE TO USE ThemeLookup, so that we get the correct resource for
4833-
// the requestedTheme, not just the value from the resources (which
4834-
// might not respect the settings' requested theme)
4835-
auto obj = ThemeLookup(res, requestedTheme, unfocusedBorderBrushKey);
4836-
_paneResources.unfocusedBorderBrush = obj.try_as<winrt::Windows::UI::Xaml::Media::SolidColorBrush>();
4837-
}
4838-
else
4839-
{
4840-
// DON'T use Transparent here - if it's "Transparent", then it won't
4841-
// be able to hittest for clicks, and then clicking on the border
4842-
// will eat focus.
4843-
_paneResources.unfocusedBorderBrush = SolidColorBrush{ Colors::Black() };
4844-
}
4845-
4846-
const auto broadcastColorKey = winrt::box_value(L"BroadcastPaneBorderColor");
4847-
if (res.HasKey(broadcastColorKey))
4848-
{
4849-
// MAKE SURE TO USE ThemeLookup
4850-
auto obj = ThemeLookup(res, requestedTheme, broadcastColorKey);
4851-
_paneResources.broadcastBorderBrush = obj.try_as<winrt::Windows::UI::Xaml::Media::SolidColorBrush>();
4852-
}
4853-
else
4854-
{
4855-
// DON'T use Transparent here - if it's "Transparent", then it won't
4856-
// be able to hittest for clicks, and then clicking on the border
4857-
// will eat focus.
4858-
_paneResources.broadcastBorderBrush = SolidColorBrush{ Colors::Black() };
48594871
}
4872+
const auto parseRelevantColorForKeyAndTheme = [this, res, requestedTheme] (const winrt::param::hstring& keyString, ThemeColor paneTheme) {
4873+
const IInspectable key = winrt::box_value(keyString);
4874+
return paneTheme ? _parseThemeColorToColor(paneTheme, res, requestedTheme, key) : _colorFromKey(res, requestedTheme, key);
4875+
};
4876+
const auto activeBrushColor = parseRelevantColorForKeyAndTheme(L"SystemAccentColor", paneActiveBorderColor);
4877+
const auto inactiveBrushColor = parseRelevantColorForKeyAndTheme(L"UnfocusedBorderBrush", paneInactiveBorderColor);
4878+
const auto broadcastBrushColor = parseRelevantColorForKeyAndTheme(L"BroadcastPaneBorderColor", broadcastBorderColor);
4879+
// For the following brushes:
4880+
// DON'T use Transparent here - if it's "Transparent", then it won't
4881+
// be able to hittest for clicks, and then clicking on the border
4882+
// will eat focus.
4883+
_paneResources.focusedBorderBrush = SolidColorBrush(activeBrushColor);
4884+
_paneResources.unfocusedBorderBrush = SolidColorBrush(inactiveBrushColor);
4885+
_paneResources.broadcastBorderBrush = SolidColorBrush(broadcastBrushColor);
48604886
}
48614887

48624888
void TerminalPage::WindowActivated(const bool activated)

‎src/cascadia/TerminalApp/TerminalPage.h‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -519,7 +519,9 @@ namespace winrt::TerminalApp::implementation
519519

520520
void _updateThemeColors();
521521
void _updateAllTabCloseButtons();
522-
void _updatePaneResources(const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme);
522+
winrt::Windows::UI::Color _colorFromKey(const winrt::Windows::UI::Xaml::ResourceDictionary& resourceDictionary, const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme, const winrt::Windows::Foundation::IInspectable& colorKey);
523+
winrt::Windows::UI::Color _parseThemeColorToColor(const winrt::Microsoft::Terminal::Settings::Model::ThemeColor& colorToCopy, const winrt::Windows::UI::Xaml::ResourceDictionary& resourceDictionary, const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme, const winrt::Windows::Foundation::IInspectable& colorKey);
524+
void _updatePaneResources(const winrt::Windows::UI::Xaml::ElementTheme& requestedTheme, const winrt::Microsoft::Terminal::Settings::Model::PaneTheme& paneTheme);
523525

524526
safe_void_coroutine _ControlCompletionsChangedHandler(const winrt::Windows::Foundation::IInspectable sender, const winrt::Microsoft::Terminal::Control::CompletionsChangedEventArgs args);
525527

‎src/cascadia/TerminalSettingsModel/MTSMSettings.h‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,8 @@ Author(s):
147147
X(winrt::Microsoft::Terminal::Settings::Model::WindowTheme, Window, "window", nullptr) \
148148
X(winrt::Microsoft::Terminal::Settings::Model::SettingsTheme, Settings, "settings", nullptr) \
149149
X(winrt::Microsoft::Terminal::Settings::Model::TabRowTheme, TabRow, "tabRow", nullptr) \
150-
X(winrt::Microsoft::Terminal::Settings::Model::TabTheme, Tab, "tab", nullptr)
150+
X(winrt::Microsoft::Terminal::Settings::Model::TabTheme, Tab, "tab", nullptr) \
151+
X(winrt::Microsoft::Terminal::Settings::Model::PaneTheme, Pane, "pane", nullptr)
151152

152153
#define MTSM_THEME_WINDOW_SETTINGS(X) \
153154
X(winrt::Windows::UI::Xaml::ElementTheme, RequestedTheme, "applicationTheme", winrt::Windows::UI::Xaml::ElementTheme::Default) \
@@ -163,6 +164,11 @@ Author(s):
163164
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, Background, "background", nullptr) \
164165
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, UnfocusedBackground, "unfocusedBackground", nullptr)
165166

167+
#define MTSM_THEME_PANE_SETTINGS(X) \
168+
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, ActiveBorderColor, "activeBorderColor", nullptr) \
169+
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, InactiveBorderColor, "inactiveBorderColor", nullptr) \
170+
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, BroadcastBorderColor, "broadcastBorderColor", nullptr)
171+
166172
#define MTSM_THEME_TAB_SETTINGS(X) \
167173
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, Background, "background", nullptr) \
168174
X(winrt::Microsoft::Terminal::Settings::Model::ThemeColor, UnfocusedBackground, "unfocusedBackground", nullptr) \

‎src/cascadia/TerminalSettingsModel/Theme.cpp‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
#include "SettingsTheme.g.h"
1313
#include "ThemeColor.g.cpp"
1414
#include "WindowTheme.g.cpp"
15+
#include "PaneTheme.g.cpp"
1516
#include "TabRowTheme.g.cpp"
1617
#include "TabTheme.g.cpp"
1718
#include "ThemePair.g.cpp"
@@ -60,6 +61,7 @@ THEME_OBJECT(WindowTheme, MTSM_THEME_WINDOW_SETTINGS);
6061
THEME_OBJECT(SettingsTheme, MTSM_THEME_SETTINGS_SETTINGS);
6162
THEME_OBJECT(TabRowTheme, MTSM_THEME_TABROW_SETTINGS);
6263
THEME_OBJECT(TabTheme, MTSM_THEME_TAB_SETTINGS);
64+
THEME_OBJECT(PaneTheme, MTSM_THEME_PANE_SETTINGS);
6365

6466
#undef THEME_SETTINGS_COPY
6567
#undef THEME_SETTINGS_TO_JSON
@@ -224,6 +226,7 @@ THEME_OBJECT_CONVERTER(winrt::Microsoft::Terminal::Settings::Model, WindowTheme,
224226
THEME_OBJECT_CONVERTER(winrt::Microsoft::Terminal::Settings::Model, SettingsTheme, MTSM_THEME_SETTINGS_SETTINGS);
225227
THEME_OBJECT_CONVERTER(winrt::Microsoft::Terminal::Settings::Model, TabRowTheme, MTSM_THEME_TABROW_SETTINGS);
226228
THEME_OBJECT_CONVERTER(winrt::Microsoft::Terminal::Settings::Model, TabTheme, MTSM_THEME_TAB_SETTINGS);
229+
THEME_OBJECT_CONVERTER(winrt::Microsoft::Terminal::Settings::Model, PaneTheme, MTSM_THEME_PANE_SETTINGS);
227230

228231
#undef THEME_SETTINGS_FROM_JSON
229232
#undef THEME_SETTINGS_TO_JSON
@@ -254,6 +257,10 @@ winrt::com_ptr<Theme> Theme::Copy() const
254257
{
255258
theme->_Tab = *winrt::get_self<implementation::TabTheme>(_Tab)->Copy();
256259
}
260+
if (_Pane)
261+
{
262+
theme->_Pane = *winrt::get_self<implementation::PaneTheme>(_Pane)->Copy();
263+
}
257264
if (_Settings)
258265
{
259266
theme->_Settings = *winrt::get_self<implementation::SettingsTheme>(_Settings)->Copy();
@@ -334,6 +341,13 @@ void Theme::LogSettingChanges(std::set<std::string>& changes, const std::string_
334341
const auto outerJsonKey = outerTabJsonKey;
335342
MTSM_THEME_TAB_SETTINGS(LOG_IF_SET)
336343
}
344+
345+
if (isPaneSet)
346+
{
347+
const auto obj = _Pane;
348+
const auto outerJsonKey = outerPaneJsonKey;
349+
MTSM_THEME_PANE_SETTINGS(LOG_IF_SET);
350+
}
337351
#undef LOG_IF_SET
338352
#undef GENERATE_SET_CHECK_AND_JSON_KEYS
339353
#pragma warning(pop)

‎src/cascadia/TerminalSettingsModel/Theme.h‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ Author(s):
2323
#include "WindowTheme.g.h"
2424
#include "TabRowTheme.g.h"
2525
#include "TabTheme.g.h"
26+
#include "PaneTheme.g.h"
2627
#include "ThemePair.g.h"
2728
#include "Theme.g.h"
2829

@@ -85,6 +86,7 @@ namespace winrt::Microsoft::Terminal::Settings::Model::implementation
8586
THEME_OBJECT(SettingsTheme, MTSM_THEME_SETTINGS_SETTINGS);
8687
THEME_OBJECT(TabRowTheme, MTSM_THEME_TABROW_SETTINGS);
8788
THEME_OBJECT(TabTheme, MTSM_THEME_TAB_SETTINGS);
89+
THEME_OBJECT(PaneTheme, MTSM_THEME_PANE_SETTINGS);
8890

8991
struct Theme : ThemeT<Theme>
9092
{

‎src/cascadia/TerminalSettingsModel/Theme.idl‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,12 @@ namespace Microsoft.Terminal.Settings.Model
7171
ThemeColor UnfocusedBackground { get; };
7272
}
7373

74+
runtimeclass PaneTheme {
75+
ThemeColor ActiveBorderColor { get; };
76+
ThemeColor InactiveBorderColor { get; };
77+
ThemeColor BroadcastBorderColor { get; };
78+
}
79+
7480
runtimeclass TabTheme {
7581
ThemeColor Background { get; };
7682
ThemeColor UnfocusedBackground { get; };
@@ -93,6 +99,9 @@ namespace Microsoft.Terminal.Settings.Model
9399
// tabRow.* Namespace
94100
TabRowTheme TabRow { get; };
95101

102+
// pane.* Namespace
103+
PaneTheme Pane { get; };
104+
96105
// tab.* Namespace
97106
TabTheme Tab { get; };
98107

0 commit comments

Comments
 (0)