From e82a90993cf130fbb898e472a9f1573fc0737a44 Mon Sep 17 00:00:00 2001
From: Evan Husted <greem@greemdev.net>
Date: Mon, 23 Dec 2024 22:32:37 -0600
Subject: [PATCH] UI: Properly disable selecting Metal on non-Mac platforms

---
 .../Views/Settings/SettingsGraphicsView.axaml  | 14 ++++++++------
 src/Ryujinx/UI/Windows/SettingsWindow.axaml.cs | 18 +++++++++---------
 2 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/src/Ryujinx/UI/Views/Settings/SettingsGraphicsView.axaml b/src/Ryujinx/UI/Views/Settings/SettingsGraphicsView.axaml
index 62c083d15..5c52d5788 100644
--- a/src/Ryujinx/UI/Views/Settings/SettingsGraphicsView.axaml
+++ b/src/Ryujinx/UI/Views/Settings/SettingsGraphicsView.axaml
@@ -33,18 +33,20 @@
                                 ToolTip.Tip="{ext:Locale SettingsTabGraphicsBackendTooltip}"
                                 Text="{ext:Locale SettingsTabGraphicsBackend}"
                                 Width="250" />
-                        <ComboBox Width="350"
-                                HorizontalContentAlignment="Left"
-                                ToolTip.Tip="{ext:Locale SettingsTabGraphicsBackendTooltip}"
-                                SelectedIndex="{Binding GraphicsBackendIndex}">
+                        <ComboBox
+                            Name="GraphicsBackendSelector"
+                            Width="350"
+                            HorizontalContentAlignment="Left"
+                            ToolTip.Tip="{ext:Locale SettingsTabGraphicsBackendTooltip}"
+                            SelectedIndex="{Binding GraphicsBackendIndex}">
                             <ComboBoxItem IsVisible="{Binding IsVulkanAvailable}">
                                 <TextBlock Text="Vulkan" />
                             </ComboBoxItem>
                             <ComboBoxItem IsEnabled="{Binding IsOpenGLAvailable}">
                                 <TextBlock Text="OpenGL" />
                             </ComboBoxItem>
-                            <ComboBoxItem IsVisible="{Binding IsMetalAvailable}">
-                                <TextBlock Text="Metal" />
+                            <ComboBoxItem IsEnabled="{Binding IsMetalAvailable}">
+                                <TextBlock Text="Metal (Experimental)" />
                             </ComboBoxItem>
                         </ComboBox>
                     </StackPanel>
diff --git a/src/Ryujinx/UI/Windows/SettingsWindow.axaml.cs b/src/Ryujinx/UI/Windows/SettingsWindow.axaml.cs
index d8c88bed8..b004d9fba 100644
--- a/src/Ryujinx/UI/Windows/SettingsWindow.axaml.cs
+++ b/src/Ryujinx/UI/Windows/SettingsWindow.axaml.cs
@@ -56,34 +56,34 @@ namespace Ryujinx.Ava.UI.Windows
             {
                 switch (navItem.Tag.ToString())
                 {
-                    case "UiPage":
+                    case nameof(UiPage):
                         UiPage.ViewModel = ViewModel;
                         NavPanel.Content = UiPage;
                         break;
-                    case "InputPage":
+                    case nameof(InputPage):
                         NavPanel.Content = InputPage;
                         break;
-                    case "HotkeysPage":
+                    case nameof(HotkeysPage):
                         NavPanel.Content = HotkeysPage;
                         break;
-                    case "SystemPage":
+                    case nameof(SystemPage):
                         SystemPage.ViewModel = ViewModel;
                         NavPanel.Content = SystemPage;
                         break;
-                    case "CpuPage":
+                    case nameof(CpuPage):
                         NavPanel.Content = CpuPage;
                         break;
-                    case "GraphicsPage":
+                    case nameof(GraphicsPage):
                         NavPanel.Content = GraphicsPage;
                         break;
-                    case "AudioPage":
+                    case nameof(AudioPage):
                         NavPanel.Content = AudioPage;
                         break;
-                    case "NetworkPage":
+                    case nameof(NetworkPage):
                         NetworkPage.ViewModel = ViewModel;
                         NavPanel.Content = NetworkPage;
                         break;
-                    case "LoggingPage":
+                    case nameof(LoggingPage):
                         NavPanel.Content = LoggingPage;
                         break;
                     default: