Code Quality: Updated existing pane toggles to prepare for upcoming Shelf feature (#16686)

Author: yaira2

src/Files.App/Actions/Show/ToggleDetailsPaneAction.cs

@@ -3,35 +3,33 @@
 
 namespace Files.App.Actions
 {
-	internal sealed class ToggleDetailsPaneAction : ObservableObject, IToggleAction
+	internal sealed class ToggleDetailsPaneAction : ObservableObject, IAction
 	{
-		private readonly InfoPaneViewModel viewModel;
+		private readonly InfoPaneViewModel infoPaneViewModel = Ioc.Default.GetRequiredService<InfoPaneViewModel>();
 		private readonly IInfoPaneSettingsService infoPaneSettingsService = Ioc.Default.GetRequiredService<IInfoPaneSettingsService>();
 
 		public string Label
-			=> "ToggleDetailsPane".GetLocalizedResource();
+			=> Strings.ToggleDetailsPane.GetLocalizedResource();
 
 		public string Description
-			=> "ToggleDetailsPaneDescription".GetLocalizedResource();
+			=> Strings.ToggleDetailsPaneDescription.GetLocalizedResource();
 
 		public RichGlyph Glyph
 			=> new(themedIconStyle: "App.ThemedIcons.PanelRight");
 
-		public HotKey HotKey
-			=> new(Keys.D, KeyModifiers.CtrlAlt);
+		public bool IsAccessibleGlobally
+			=> false;
 
-		public bool IsOn
-			=> viewModel.IsEnabled;
+		public bool IsExecutable
+			=> infoPaneViewModel.IsEnabled;
 
 		public ToggleDetailsPaneAction()
 		{
-			viewModel = Ioc.Default.GetRequiredService<InfoPaneViewModel>();
-			viewModel.PropertyChanged += ViewModel_PropertyChanged;
+			infoPaneViewModel.PropertyChanged += ViewModel_PropertyChanged;
 		}
 
 		public Task ExecuteAsync(object? parameter = null)
 		{
-			viewModel.IsEnabled = true;
 			infoPaneSettingsService.SelectedTab = InfoPaneTabs.Details;
 
 			return Task.CompletedTask;
@@ -40,7 +38,7 @@ public Task ExecuteAsync(object? parameter = null)
 		private void ViewModel_PropertyChanged(object? sender, PropertyChangedEventArgs e)
 		{
 			if (e.PropertyName is nameof(InfoPaneViewModel.IsEnabled))
-				OnPropertyChanged(nameof(IsOn));
+				OnPropertyChanged(nameof(IsExecutable));
 		}
 	}
 }

src/Files.App/Actions/Show/TogglePreviewPaneAction.cs

@@ -3,35 +3,33 @@
 
 namespace Files.App.Actions
 {
-	internal sealed class TogglePreviewPaneAction : ObservableObject, IToggleAction
+	internal sealed class TogglePreviewPaneAction : ObservableObject, IAction
 	{
-		private readonly InfoPaneViewModel viewModel;
+		private readonly InfoPaneViewModel infoPaneViewModel = Ioc.Default.GetRequiredService<InfoPaneViewModel>();
 		private readonly IInfoPaneSettingsService infoPaneSettingsService = Ioc.Default.GetRequiredService<IInfoPaneSettingsService>();
 
 		public string Label
-			=> "TogglePreviewPane".GetLocalizedResource();
+			=> Strings.TogglePreviewPane.GetLocalizedResource();
 
 		public string Description
-			=> "TogglePreviewPaneDescription".GetLocalizedResource();
+			=> Strings.TogglePreviewPaneDescription.GetLocalizedResource();
 
 		public RichGlyph Glyph
 			=> new(themedIconStyle: "App.ThemedIcons.PanelRight");
 
-		public HotKey HotKey
-			=> new(Keys.P, KeyModifiers.CtrlAlt);
+		public bool IsAccessibleGlobally
+			=> false;
 
-		public bool IsOn
-			=> viewModel.IsEnabled;
+		public bool IsExecutable
+			=> infoPaneViewModel.IsEnabled;
 
 		public TogglePreviewPaneAction()
 		{
-			viewModel = Ioc.Default.GetRequiredService<InfoPaneViewModel>();
-			viewModel.PropertyChanged += ViewModel_PropertyChanged;
+			infoPaneViewModel.PropertyChanged += ViewModel_PropertyChanged;
 		}
 
 		public Task ExecuteAsync(object? parameter = null)
 		{
-			viewModel.IsEnabled = true;
 			infoPaneSettingsService.SelectedTab = InfoPaneTabs.Preview;
 
 			return Task.CompletedTask;
@@ -40,7 +38,7 @@ public Task ExecuteAsync(object? parameter = null)
 		private void ViewModel_PropertyChanged(object? sender, PropertyChangedEventArgs e)
 		{
 			if (e.PropertyName is nameof(InfoPaneViewModel.IsEnabled))
-				OnPropertyChanged(nameof(IsOn));
+				OnPropertyChanged(nameof(IsExecutable));
 		}
 	}
 }