From 0b58355ef4b7b81b7b9d1a3ca99810dbcf9eaa6c Mon Sep 17 00:00:00 2001 From: Pankaj Bhojwani Date: Tue, 2 Nov 2021 10:59:52 -0700 Subject: [PATCH 1/6] testing --- src/cascadia/TerminalApp/App.xaml | 2 +- .../CommonResources.xaml | 4 +- .../SettingContainerStyle.xaml | 42 +++++++++++++------ 3 files changed, 32 insertions(+), 16 deletions(-) diff --git a/src/cascadia/TerminalApp/App.xaml b/src/cascadia/TerminalApp/App.xaml index 251028dee..3a880ead3 100644 --- a/src/cascadia/TerminalApp/App.xaml +++ b/src/cascadia/TerminalApp/App.xaml @@ -21,7 +21,7 @@ + ControlsResourcesVersion="Version2" /> diff --git a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml index 717e6e808..0597bcab9 100644 --- a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml +++ b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml @@ -38,19 +38,35 @@ - - - - - - - - - + + + + + + + + + + + + + From ba4faec04fa7761742c155bbd1345de0341e3d0f Mon Sep 17 00:00:00 2001 From: Pankaj Bhojwani Date: Wed, 3 Nov 2021 15:50:29 -0700 Subject: [PATCH 2/6] got 2 styles that kinda work I guess --- .../TerminalSettingsEditor/Appearances.xaml | 4 - .../CommonResources.xaml | 22 +- .../GlobalAppearance.xaml | 1 - .../TerminalSettingsEditor/Interaction.xaml | 3 +- .../TerminalSettingsEditor/Launch.xaml | 7 +- .../TerminalSettingsEditor/Profiles.xaml | 7 - .../SettingContainer.cpp | 17 + .../SettingContainerStyle.xaml | 501 +++++++++++++++++- 8 files changed, 499 insertions(+), 63 deletions(-) diff --git a/src/cascadia/TerminalSettingsEditor/Appearances.xaml b/src/cascadia/TerminalSettingsEditor/Appearances.xaml index 497b7349a..a8f9f2ac2 100644 --- a/src/cascadia/TerminalSettingsEditor/Appearances.xaml +++ b/src/cascadia/TerminalSettingsEditor/Appearances.xaml @@ -45,7 +45,6 @@ @@ -170,7 +169,6 @@ @@ -211,7 +209,6 @@ @@ -450,7 +447,6 @@ diff --git a/src/cascadia/TerminalSettingsEditor/CommonResources.xaml b/src/cascadia/TerminalSettingsEditor/CommonResources.xaml index 6f5fdf4f0..36cbb7980 100644 --- a/src/cascadia/TerminalSettingsEditor/CommonResources.xaml +++ b/src/cascadia/TerminalSettingsEditor/CommonResources.xaml @@ -11,26 +11,6 @@ - - - - - - - - - diff --git a/src/cascadia/TerminalSettingsEditor/GlobalAppearance.xaml b/src/cascadia/TerminalSettingsEditor/GlobalAppearance.xaml index 7361ec7cb..002b7eda8 100644 --- a/src/cascadia/TerminalSettingsEditor/GlobalAppearance.xaml +++ b/src/cascadia/TerminalSettingsEditor/GlobalAppearance.xaml @@ -28,7 +28,6 @@ diff --git a/src/cascadia/TerminalSettingsEditor/Interaction.xaml b/src/cascadia/TerminalSettingsEditor/Interaction.xaml index 8bdf829f6..273d67eed 100644 --- a/src/cascadia/TerminalSettingsEditor/Interaction.xaml +++ b/src/cascadia/TerminalSettingsEditor/Interaction.xaml @@ -27,8 +27,7 @@ - + diff --git a/src/cascadia/TerminalSettingsEditor/Launch.xaml b/src/cascadia/TerminalSettingsEditor/Launch.xaml index 67a4a44d0..de7f3136d 100644 --- a/src/cascadia/TerminalSettingsEditor/Launch.xaml +++ b/src/cascadia/TerminalSettingsEditor/Launch.xaml @@ -37,8 +37,8 @@ - + - + diff --git a/src/cascadia/TerminalSettingsEditor/Profiles.xaml b/src/cascadia/TerminalSettingsEditor/Profiles.xaml index 230cfc2e8..d11424434 100644 --- a/src/cascadia/TerminalSettingsEditor/Profiles.xaml +++ b/src/cascadia/TerminalSettingsEditor/Profiles.xaml @@ -66,7 +66,6 @@ so the name will always be overridden. --> @@ -75,7 +74,6 @@ @@ -249,7 +246,6 @@ @@ -271,7 +267,6 @@ @@ -288,7 +283,6 @@ @@ -405,7 +399,6 @@ diff --git a/src/cascadia/TerminalSettingsEditor/SettingContainer.cpp b/src/cascadia/TerminalSettingsEditor/SettingContainer.cpp index 7d53785c6..0dac71246 100644 --- a/src/cascadia/TerminalSettingsEditor/SettingContainer.cpp +++ b/src/cascadia/TerminalSettingsEditor/SettingContainer.cpp @@ -72,6 +72,23 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation void SettingContainer::OnApplyTemplate() { + if (const auto& child{ GetTemplateChild(L"ExpanderButton") }) + { + if (const auto& toggleButton{ child.try_as() }) + { + toggleButton.Click([=](auto&&, auto&&) { + if (const auto& expanderChild{ GetTemplateChild(L"MainContentContainer") }) + { + if (const auto& expander{ expanderChild.try_as() }) + { + const auto newVisibility = expander.Visibility() == Visibility::Visible ? Visibility::Collapsed : Visibility::Visible; + expander.Visibility(newVisibility); + } + } + }); + } + } + if (const auto& child{ GetTemplateChild(L"ResetButton") }) { if (const auto& button{ child.try_as() }) diff --git a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml index 0597bcab9..0fde41dda 100644 --- a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml +++ b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml @@ -9,6 +9,48 @@ xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" mc:Ignorable="d"> + + + + + + + + + + + + + + + + + + + + + + - @@ -29,6 +19,16 @@ TargetType="TextBlock"> + #B3FFFFFF + #0F000000 + + + + + + 96 + 1 + 4 + #FF0000 + #FF0000 + + + + + + 96 + 2 + 4 + #0DFFFFFF + #19000000 + + + + + + 96 + 1 + 4 @@ -73,28 +93,59 @@ + + + + + + - - - - + + From 81d0c5c2d7a28ecbaee9b2ce9f1337d64ce79305 Mon Sep 17 00:00:00 2001 From: Pankaj Bhojwani Date: Tue, 9 Nov 2021 15:10:35 -0800 Subject: [PATCH 5/6] remove remnants of previous implementation, format --- .../TerminalSettingsEditor/Appearances.xaml | 10 +- .../CommonResources.xaml | 2 +- .../TerminalSettingsEditor/Interaction.xaml | 4 +- .../TerminalSettingsEditor/Launch.xaml | 4 +- .../TerminalSettingsEditor/Profiles.xaml | 20 ++-- .../SettingContainerStyle.xaml | 107 ++++++------------ 6 files changed, 53 insertions(+), 94 deletions(-) diff --git a/src/cascadia/TerminalSettingsEditor/Appearances.xaml b/src/cascadia/TerminalSettingsEditor/Appearances.xaml index 244140b22..b50416f20 100644 --- a/src/cascadia/TerminalSettingsEditor/Appearances.xaml +++ b/src/cascadia/TerminalSettingsEditor/Appearances.xaml @@ -210,11 +210,11 @@ + SettingOverrideSource="{x:Bind Appearance.BackgroundImagePathOverrideSource, Mode=OneWay}" + Style="{StaticResource ExpanderSettingContainerStyle}"> diff --git a/src/cascadia/TerminalSettingsEditor/CommonResources.xaml b/src/cascadia/TerminalSettingsEditor/CommonResources.xaml index 36cbb7980..20304fc18 100644 --- a/src/cascadia/TerminalSettingsEditor/CommonResources.xaml +++ b/src/cascadia/TerminalSettingsEditor/CommonResources.xaml @@ -36,7 +36,7 @@ diff --git a/src/cascadia/TerminalSettingsEditor/Interaction.xaml b/src/cascadia/TerminalSettingsEditor/Interaction.xaml index de64f99c8..1fba2248b 100644 --- a/src/cascadia/TerminalSettingsEditor/Interaction.xaml +++ b/src/cascadia/TerminalSettingsEditor/Interaction.xaml @@ -52,8 +52,8 @@ + CurrentValue="{x:Bind State.Globals.WordDelimiters, Mode=OneWay}" + Style="{StaticResource ExpanderSettingContainerStyle}"> diff --git a/src/cascadia/TerminalSettingsEditor/Launch.xaml b/src/cascadia/TerminalSettingsEditor/Launch.xaml index c5e770a47..a58730b6e 100644 --- a/src/cascadia/TerminalSettingsEditor/Launch.xaml +++ b/src/cascadia/TerminalSettingsEditor/Launch.xaml @@ -134,8 +134,8 @@ + Style="{StaticResource ExpanderSettingContainerStyle}" + Visibility="{x:Bind ShowFirstWindowPreference}"> @@ -77,10 +77,10 @@ + SettingOverrideSource="{x:Bind State.Profile.StartingDirectoryOverrideSource, Mode=OneWay}" + Style="{StaticResource ExpanderSettingContainerStyle}"> + HasSettingValue="{x:Bind State.Profile.HasIcon, Mode=OneWay}" + SettingOverrideSource="{x:Bind State.Profile.IconOverrideSource, Mode=OneWay}" + Style="{StaticResource ExpanderSettingContainerStyle}"> + SettingOverrideSource="{x:Bind State.Profile.TabTitleOverrideSource, Mode=OneWay}" + Style="{StaticResource ExpanderSettingContainerStyle}"> diff --git a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml index 3d88cbfe1..bfad7745c 100644 --- a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml +++ b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml @@ -12,63 +12,27 @@ - - - #B3FFFFFF - #0F000000 - - - - - - 96 - 1 - 4 + - - - #0DFFFFFF - #19000000 - - - - - - 96 - 1 - 4 + @@ -94,7 +58,8 @@ - - - - diff --git a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml index bfad7745c..155fb792e 100644 --- a/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml +++ b/src/cascadia/TerminalSettingsEditor/SettingContainerStyle.xaml @@ -67,9 +67,7 @@ - -