Skip to content

Commit 5f4087f

Browse files
committed
finish exorcising SettingsTab
1 parent e82c627 commit 5f4087f

File tree

8 files changed

+6
-224
lines changed

8 files changed

+6
-224
lines changed

src/cascadia/TerminalApp/SettingsTab.cpp

Lines changed: 0 additions & 131 deletions
This file was deleted.

src/cascadia/TerminalApp/SettingsTab.h

Lines changed: 0 additions & 43 deletions
This file was deleted.

src/cascadia/TerminalApp/SettingsTab.idl

Lines changed: 0 additions & 12 deletions
This file was deleted.

src/cascadia/TerminalApp/TabManagement.cpp

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
#include "TabRowControl.h"
2020
#include "ColorHelper.h"
2121
#include "DebugTapConnection.h"
22-
#include "SettingsTab.h"
2322
#include "..\TerminalSettingsModel\FileUtils.h"
2423

2524
#include <shlobj.h>
@@ -794,14 +793,6 @@ namespace winrt::TerminalApp::implementation
794793
}
795794
}
796795
}
797-
else if (auto index{ _GetFocusedTabIndex() })
798-
{
799-
const auto tab{ _tabs.GetAt(*index) };
800-
if (tab.try_as<TerminalApp::SettingsTab>())
801-
{
802-
_HandleCloseTabRequested(tab);
803-
}
804-
}
805796
}
806797

807798
// Method Description:

src/cascadia/TerminalApp/TerminalAppLib.vcxproj

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,6 @@
8888
<DependentUpon>PaletteItemTemplateSelector.idl</DependentUpon>
8989
<SubType>Code</SubType>
9090
</ClInclude>
91-
<ClInclude Include="SettingsTab.h">
92-
<DependentUpon>SettingsTab.idl</DependentUpon>
93-
</ClInclude>
9491
<ClInclude Include="PaletteItem.h" />
9592
<ClInclude Include="TabBase.h">
9693
<DependentUpon>TabBase.idl</DependentUpon>
@@ -194,9 +191,6 @@
194191
<DependentUpon>PaletteItemTemplateSelector.idl</DependentUpon>
195192
<SubType>Code</SubType>
196193
</ClCompile>
197-
<ClCompile Include="SettingsTab.cpp">
198-
<DependentUpon>SettingsTab.idl</DependentUpon>
199-
</ClCompile>
200194
<ClCompile Include="PaletteItem.cpp" />
201195
<ClCompile Include="TabBase.cpp">
202196
<DependentUpon>TabBase.idl</DependentUpon>
@@ -312,7 +306,6 @@
312306
<Midl Include="PaletteItemTemplateSelector.idl">
313307
<SubType>Designer</SubType>
314308
</Midl>
315-
<Midl Include="SettingsTab.idl" />
316309
<Midl Include="PaletteItem.idl" />
317310
<Midl Include="ShortcutActionDispatch.idl" />
318311
<Midl Include="AppKeyBindings.idl" />

src/cascadia/TerminalApp/TerminalAppLib.vcxproj.filters

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,6 @@
2424
<ClCompile Include="Tab.cpp">
2525
<Filter>tab</Filter>
2626
</ClCompile>
27-
<ClCompile Include="SettingsTab.cpp">
28-
<Filter>tab</Filter>
29-
</ClCompile>
3027
<ClCompile Include="FilteredCommand.cpp">
3128
<Filter>commandPalette</Filter>
3229
</ClCompile>
@@ -64,9 +61,6 @@
6461
<ClInclude Include="Tab.h">
6562
<Filter>tab</Filter>
6663
</ClInclude>
67-
<ClInclude Include="SettingsTab.h">
68-
<Filter>tab</Filter>
69-
</ClInclude>
7064
<ClInclude Include="FilteredCommand.h">
7165
<Filter>commandPalette</Filter>
7266
</ClInclude>
@@ -99,9 +93,6 @@
9993
<Filter>settings</Filter>
10094
</Midl>
10195
<Midl Include="IDirectKeyListener.idl" />
102-
<Midl Include="SettingsTab.idl">
103-
<Filter>tab</Filter>
104-
</Midl>
10596
<Midl Include="TerminalTab.idl">
10697
<Filter>tab</Filter>
10798
</Midl>
@@ -187,4 +178,4 @@
187178
<Filter>app</Filter>
188179
</ApplicationDefinition>
189180
</ItemGroup>
190-
</Project>
181+
</Project>

src/cascadia/TerminalApp/TerminalPage.cpp

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
#include "App.h"
2222
#include "ColorHelper.h"
2323
#include "DebugTapConnection.h"
24-
#include "SettingsTab.h"
2524
#include "SettingsPaneContent.h"
2625
#include "TabRowControl.h"
2726
#include "Utils.h"
@@ -3341,10 +3340,6 @@ namespace winrt::TerminalApp::implementation
33413340
// Force the TerminalTab to re-grab its currently active control's title.
33423341
terminalTab->UpdateTitle();
33433342
}
3344-
else if (auto settingsTab = tab.try_as<TerminalApp::SettingsTab>())
3345-
{
3346-
settingsTab.UpdateSettings(_settings);
3347-
}
33483343

33493344
auto tabImpl{ winrt::get_self<TabBase>(tab) };
33503345
tabImpl->SetActionMap(_settings.ActionMap());
@@ -4612,13 +4607,12 @@ namespace winrt::TerminalApp::implementation
46124607
til::color bgColor = backgroundSolidBrush.Color();
46134608

46144609
Media::Brush terminalBrush{ nullptr };
4615-
if (const auto& control{ _GetActiveControl() })
4610+
if (const auto tab{ _GetFocusedTabImpl() })
46164611
{
4617-
terminalBrush = control.BackgroundBrush();
4618-
}
4619-
else if (const auto& settingsTab{ _GetFocusedTab().try_as<TerminalApp::SettingsTab>() })
4620-
{
4621-
terminalBrush = settingsTab.Content().try_as<Settings::Editor::MainPage>().BackgroundBrush();
4612+
if (const auto& pane{ tab->GetActivePane() })
4613+
{
4614+
terminalBrush = pane->GetContent().BackgroundBrush();
4615+
}
46224616
}
46234617

46244618
if (_settings.GlobalSettings().UseAcrylicInTabRow())

src/cascadia/TerminalApp/dll/TerminalApp.vcxproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@
3636
<ClInclude Include="../TabRowControl.h" />
3737
<ClInclude Include="../App.h" />
3838
<ClInclude Include="../TerminalTab.h" />
39-
<ClInclude Include="../SettingsTab.h" />
4039
</ItemGroup>
4140
<!-- ========================= Cpp Files ======================== -->
4241
<ItemGroup>

0 commit comments

Comments
 (0)