Remove Mica workaround

No longer needed for Windows App SDK 1.3.230502000: microsoft/microsoft-ui-xaml#8416 (comment)
JasonWei512 · May 10, 2023 · f9706dc · f9706dc
1 parent 115d2ee
commit f9706dc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/src/EnergyStarX/MainWindow.xaml b/src/EnergyStarX/MainWindow.xaml
@@ -10,4 +10,8 @@
     MinWidth="720"
     MinHeight="480"
     PersistenceId="MainWindow"
-    mc:Ignorable="d" />
+    mc:Ignorable="d">
+    <windowex:WindowEx.SystemBackdrop>
+        <MicaBackdrop />
+    </windowex:WindowEx.SystemBackdrop>
+</windowex:WindowEx>
diff --git a/src/EnergyStarX/Services/ActivationService.cs b/src/EnergyStarX/Services/ActivationService.cs
@@ -5,7 +5,6 @@
 
 using Microsoft.UI.Xaml;
 using Microsoft.UI.Xaml.Controls;
-using Microsoft.UI.Xaml.Media;
 using Microsoft.Windows.AppLifecycle;
 
 namespace EnergyStarX.Services;
@@ -50,11 +49,6 @@ public async Task Activate(object activationArgs)
         {
             shell = App.GetService<ShellPage>();
             App.MainWindow.Content = shell ?? new Frame();
-
-            // Windows App SDK 1.3's new SystemBackdrop API requires the window to have content.
-            // If I set MainWindow.SystemBackdrop when MainWindow.Content is null, the app may crash on launch.
-            // https://github.com/microsoft/WindowsAppSDK/discussions/3406#discussioncomment-4955796
-            App.MainWindow.SystemBackdrop = new MicaBackdrop();
         }
 
         // Handle activation via ActivationHandlers.