Merge pull request #4381 from DavidGBrett/plugin-settings-context-menu-option

[FEATURE] Plugin settings context menu option

Author: Jack251970

Flow.Launcher.Core/Plugin/JsonRPCV2Models/JsonRPCPublicAPI.cs

@@ -94,6 +94,11 @@ public void OpenSettingDialog()
             _api.OpenSettingDialog();
         }
 
+        public bool OpenPluginSettingsWindow(string pluginId)
+        {
+            return _api.OpenPluginSettingsWindow(pluginId);
+        }
+
         public string GetTranslation(string key)
         {
             return _api.GetTranslation(key);

Flow.Launcher.Infrastructure/Constant.cs

@@ -33,6 +33,7 @@ public static class Constant
         public static readonly string LoadingImgIcon = Path.Combine(ImagesDirectory, "loading.png");
         public static readonly string ImageIcon = Path.Combine(ImagesDirectory, "image.png");
         public static readonly string HistoryIcon = Path.Combine(ImagesDirectory, "history.png");
+        public static readonly string SettingsIcon = Path.Combine(ImagesDirectory, "settings.png");
 
         public static string PythonPath;
         public static string NodePath;

Flow.Launcher.Plugin/Interfaces/IPublicAPI.cs

@@ -162,6 +162,15 @@ public interface IPublicAPI
         /// </summary>
         void OpenSettingDialog();
 
+
+        /// <summary>
+        /// Open plugin setting window for a specific plugin.
+        /// Reuses the existing window when that plugin's settings window is already open.
+        /// </summary>
+        /// <param name="pluginId">ID of the plugin whose settings window should be opened</param>
+        /// <returns>True if the plugin settings window was successfully opened or reused; false otherwise</returns>
+        bool OpenPluginSettingsWindow(string pluginId);
+
         /// <summary>
         /// Get translation of current language
         /// You need to implement IPluginI18n if you want to support multiple languages for your plugin

Flow.Launcher/Helper/SingletonWindowOpener.cs

@@ -11,30 +11,6 @@ public static T Open<T>(params object[] args) where T : Window
         var window = Application.Current.Windows.OfType<Window>().FirstOrDefault(x => x.GetType() == typeof(T))
                      ?? (T)Activator.CreateInstance(typeof(T), args);
 
-        // Fix UI bug
-        // Add `window.WindowState = WindowState.Normal`
-        // If only use `window.Show()`, Settings-window doesn't show when minimized in taskbar 
-        // Not sure why this works tho
-        // Probably because, when `.Show()` fails, `window.WindowState == Minimized` (not `Normal`) 
-        // https://stackoverflow.com/a/59719760/4230390
-        // Ensure the window is not minimized before showing it
-        if (window.WindowState == WindowState.Minimized)
-        {
-            window.WindowState = WindowState.Normal;
-        }
-
-        // Ensure the window is visible
-        if (!window.IsVisible)
-        {
-            window.Show();
-        }
-        else
-        {
-            window.Activate(); // Bring the window to the foreground if already open
-        }
-
-        window.Focus();
-
-        return (T)window;
+        return WindowVisibilityHelper.ShowOrActivate((T)window);
     }
 }

Flow.Launcher/Helper/WindowVisibilityHelper.cs

@@ -0,0 +1,35 @@
+using System.Windows;
+
+namespace Flow.Launcher.Helper;
+
+public static class WindowVisibilityHelper
+{
+    public static T ShowOrActivate<T>(T window) where T : Window
+    {
+        // Fix UI bug
+        // Add `window.WindowState = WindowState.Normal`
+        // If only use `window.Show()`, Settings-window doesn't show when minimized in taskbar 
+        // Not sure why this works tho
+        // Probably because, when `.Show()` fails, `window.WindowState == Minimized` (not `Normal`) 
+        // https://stackoverflow.com/a/59719760/4230390
+        // Ensure the window is not minimized before showing it
+        if (window.WindowState == WindowState.Minimized)
+        {
+            window.WindowState = WindowState.Normal;
+        }
+
+        // Ensure the window is visible
+        if (!window.IsVisible)
+        {
+            window.Show();
+        }
+        else
+        {
+            window.Activate(); // Bring the window to the foreground if already open
+        }
+
+        window.Focus();
+
+        return (T)window;
+    }
+}

Flow.Launcher/Languages/en.xaml

@@ -484,6 +484,10 @@
     <system:String x:Key="LogLevelDEBUG">Debug</system:String>
     <system:String x:Key="settingWindowFontTitle">Setting Window Font</system:String>
 
+    <!--  Plugin Settings Window  -->
+    <system:String x:Key="pluginSettingsWindowTitle">Settings: {0}</system:String>
+    <system:String x:Key="pluginSettingsWindowOpenFailed">Failed to open plugin settings window</system:String>
+
     <!--  Release Notes Window  -->
     <system:String x:Key="seeMoreReleaseNotes">See more release notes on GitHub</system:String>
     <system:String x:Key="checkNetworkConnectionTitle">Failed to fetch release notes</system:String>