refactor: Centralize image opening logic

Refactored the image opening logic by creating a centralized `ShowImageBrowser` method in `UiUtil` class. This reduces redundancy and improves maintainability across the `ProSearchResultViewModel`, `PosterPickerViewModel`, and `ManualExplorerViewModel` classes.

Author: DineshSolanki

FoliCon/Modules/UI/UIUtil.cs

@@ -0,0 +1,19 @@
+using NLog;
+using Logger = NLog.Logger;
+
+namespace FoliCon.Modules.UI;
+
+public static class UiUtil
+{
+    private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
+    public static void ShowImageBrowser(string imageLocation)
+    {
+        Logger.Trace("Opening Image {Image}", imageLocation);
+        var browser = new ImageBrowser(imageLocation)
+        {
+            ShowTitle = false,
+            IsFullScreen = true
+        };
+        browser.Show();
+    }
+}

FoliCon/ViewModels/ManualExplorerViewModel.cs

@@ -2,6 +2,7 @@
 using FoliCon.Models.Api;
 using FoliCon.Models.Data;
 using FoliCon.Modules.DeviantArt;
+using FoliCon.Modules.UI;
 using NLog;
 
 namespace FoliCon.ViewModels;
@@ -14,7 +15,7 @@ public ManualExplorerViewModel()
 	{
 		Directory = [];
 		PickCommand = new DelegateCommand<object>(PickMethod);
-		OpenImageCommand = new DelegateCommand<object>(OpenImageMethod);
+		OpenImageCommand = new DelegateCommand<object>((link)=> UiUtil.ShowImageBrowser(link as string));
 		CancelCommand = new DelegateCommand(CancelMethod);
 	}
 
@@ -49,19 +50,6 @@ private void CancelMethod()
 		_cts.Cancel();
 	}
 
-
-	private static void OpenImageMethod(object parameter)
-	{
-		Logger.Debug("Opening Image {Image}", parameter);
-		var link = (string)parameter;
-		var browser = new ImageBrowser(link)
-		{
-			ShowTitle = false,
-			IsFullScreen = true
-		};
-		browser.Show();
-	}
-	
 	#region DialogMethods
 
 	public event Action<IDialogResult> RequestClose;

FoliCon/ViewModels/PosterPickerViewModel.cs

@@ -60,13 +60,7 @@ private void OpenImageMethod(DArtImageList selectedImage)
         link = Result.MediaType == MediaTypes.Game
             ? IgdbDataTransformer.GetPosterUrl(selectedImage.DeviationId, ImageSize.HD720)
             : link;
-        Logger.Info("Opening Image: {Link}", link);
-        var browser = new ImageBrowser(link)
-        {
-            ShowTitle = false,
-            IsFullScreen = true
-        };
-        browser.Show();
+        UiUtil.ShowImageBrowser(link);
     }
 
     protected virtual void CloseDialog(string parameter)

FoliCon/ViewModels/ProSearchResultViewModel.cs

@@ -86,25 +86,13 @@ public ProSearchResultViewModel(IDialogService dialogService)
         ImageUrl = [];
         StopSearchCommand = new DelegateCommand(delegate { StopSearch = true; });
         PickCommand = new DelegateCommand<object>(PickMethod);
-        OpenImageCommand = new DelegateCommand<object>(OpenImageMethod);
+        OpenImageCommand = new DelegateCommand<object>((link)=> UiUtil.ShowImageBrowser(link as string));
         ExtractManuallyCommand = new DelegateCommand<object>(ExtractManually);
         SkipCommand = new DelegateCommand(SkipMethod);
         SearchAgainCommand = new DelegateCommand(PrepareForSearch);
         _dialogService = dialogService;
     }
 
-    private static void OpenImageMethod(object parameter)
-    {
-        Logger.Debug("Opening Image {Image}", parameter);
-        var link = (string)parameter;
-        var browser = new ImageBrowser(link)
-        {
-            ShowTitle = false,
-            IsFullScreen = true
-        };
-        browser.Show();
-    }
-
     private void ExtractManually(object parameter)
     {
         Logger.Debug("Extracting manually from Deviation ID {DeviationId}", parameter);