From e6f70a0c82508057339cfed8742ec6e675ec57d5 Mon Sep 17 00:00:00 2001
From: pizzaboxer <pizzaboxer@pizzaboxer.xyz>
Date: Fri, 6 Sep 2024 09:59:49 +0100
Subject: [PATCH] Remove option for disabling fullscreen optimizations

The option was only introduced because Roblox's installation path wasn't static. Now that it is, this option doesn't need to exist anymore, and it isn't an issue for people to just set it manually now.
---
 Bloxstrap/Models/Settings.cs                          | 1 -
 Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml    | 8 --------
 Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml.cs | 8 +-------
 Bloxstrap/UI/ViewModels/Settings/ModsViewModel.cs     | 6 ------
 4 files changed, 1 insertion(+), 22 deletions(-)

diff --git a/Bloxstrap/Models/Settings.cs b/Bloxstrap/Models/Settings.cs
index 11ce9c3..ebe594f 100644
--- a/Bloxstrap/Models/Settings.cs
+++ b/Bloxstrap/Models/Settings.cs
@@ -25,6 +25,5 @@ namespace Bloxstrap.Models
 
         // mod preset configuration
         public bool UseDisableAppPatch { get; set; } = false;
-        public bool DisableFullscreenOptimizations { get; set; } = false;
     }
 }
diff --git a/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml b/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml
index bb0e06f..af34a12 100644
--- a/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml
+++ b/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml
@@ -93,13 +93,5 @@
                 <ui:Button Icon="Delete16" Content="{x:Static resources:Strings.Menu_Mods_Misc_CustomFont_Remove}" Appearance="Danger" Command="{Binding ManageCustomFontCommand}" Visibility="{Binding DeleteCustomFontVisibility, Mode=OneWay}" />
             </StackPanel>
         </controls:OptionControl>
-
-        <controls:OptionControl 
-            Header="{x:Static resources:Strings.Menu_Mods_Misc_DisableFullscreenOptimisations_Title}" 
-            Description="{x:Static resources:Strings.Menu_Mods_Misc_DisableFullscreenOptimisations_Description}"
-            HelpLink="https://devblogs.microsoft.com/directx/demystifying-full-screen-optimizations/"
-            x:Name="FullscreenOptimizationsToggle">
-            <ui:ToggleSwitch IsChecked="{Binding DisableFullscreenOptimizations, Mode=TwoWay}" />
-        </controls:OptionControl>
     </StackPanel>
 </ui:UiPage>
diff --git a/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml.cs b/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml.cs
index 4e7e204..dc82f08 100644
--- a/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml.cs
+++ b/Bloxstrap/UI/Elements/Settings/Pages/ModsPage.xaml.cs
@@ -1,6 +1,4 @@
-using System.Windows;
-
-using Bloxstrap.UI.ViewModels.Settings;
+using Bloxstrap.UI.ViewModels.Settings;
 
 namespace Bloxstrap.UI.Elements.Settings.Pages
 {
@@ -13,10 +11,6 @@ namespace Bloxstrap.UI.Elements.Settings.Pages
         {
             DataContext = new ModsViewModel();
             InitializeComponent();
-
-            // fullscreen optimizations were only added in windows 10 build 17093
-            if (Environment.OSVersion.Version.Build < 17093)
-                this.FullscreenOptimizationsToggle.Visibility = Visibility.Collapsed;
         }
     }
 }
diff --git a/Bloxstrap/UI/ViewModels/Settings/ModsViewModel.cs b/Bloxstrap/UI/ViewModels/Settings/ModsViewModel.cs
index f36b897..0e85f6c 100644
--- a/Bloxstrap/UI/ViewModels/Settings/ModsViewModel.cs
+++ b/Bloxstrap/UI/ViewModels/Settings/ModsViewModel.cs
@@ -95,11 +95,5 @@ namespace Bloxstrap.UI.ViewModels.Settings
         });
 
         public FontModPresetTask TextFontTask { get; } = new();
-
-        public bool DisableFullscreenOptimizations
-        {
-            get => App.Settings.Prop.DisableFullscreenOptimizations;
-            set => App.Settings.Prop.DisableFullscreenOptimizations = value;
-        }
     }
 }