Refactor ShellSettings with Binding & Fix IsEnabled logic

Plugins/Flow.Launcher.Plugin.Shell/Converters/LeaveShellOpenOrCloseShellAfterPressEnabledConverter.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Globalization;
+using System.Windows.Data;
+
+namespace Flow.Launcher.Plugin.Shell.Converters;
+
+public class LeaveShellOpenOrCloseShellAfterPressEnabledConverter : IMultiValueConverter
+{
+    public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture)
+    {
+        if (
+            values.Length != 2 ||
+            values[0] is not bool closeShellAfterPressOrLeaveShellOpen ||
+            values[1] is not Shell shell
+        )
+            return Binding.DoNothing;
+
+        return (!closeShellAfterPressOrLeaveShellOpen) && shell != Shell.RunCommand;
+    }
+
+    public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)
+    {
+        throw new NotImplementedException();
+    }
+}

Plugins/Flow.Launcher.Plugin.Shell/Main.cs

@@ -6,6 +6,7 @@
 using System.Linq;
 using System.Threading.Tasks;
 using Flow.Launcher.Plugin.SharedCommands;
+using Flow.Launcher.Plugin.Shell.Views;
 using WindowsInput;
 using WindowsInput.Native;
 using Control = System.Windows.Controls.Control;
@@ -45,14 +46,14 @@
                     string basedir = null;
                     string dir = null;
                     string excmd = Environment.ExpandEnvironmentVariables(cmd);
                     if (Directory.Exists(excmd) && (cmd.EndsWith('/') || cmd.EndsWith('\\')))
                     {
                         basedir = excmd;
                         dir = cmd;
                     }
                     else if (Directory.Exists(Path.GetDirectoryName(excmd) ?? string.Empty))
                     {
                         basedir = Path.GetDirectoryName(excmd);
                         var dirName = Path.GetDirectoryName(cmd);
                         dir = (dirName.EndsWith('/') || dirName.EndsWith('\\')) ? dirName : cmd[..(dirName.Length + 1)];
                     }
@@ -129,7 +130,7 @@
                 }).Where(o => o != null);
 
             if (_settings.ShowOnlyMostUsedCMDs)
                 return [.. history.Take(_settings.ShowOnlyMostUsedCMDsNumber)];
 
             return [.. history];
         }
@@ -181,8 +182,8 @@
                     CopyText = m.Key
                 });
 
             if (_settings.ShowOnlyMostUsedCMDs)
                 return [.. history.Take(_settings.ShowOnlyMostUsedCMDsNumber)];
 
             return [.. history];
         }
@@ -271,7 +272,7 @@
                         }
                         else
                         {
                             info.FileName = "pwsh.exe";
                         }
                         if (_settings.LeaveShellOpen)
                         {
@@ -391,11 +392,11 @@
             {
                 if (keyevent == (int)KeyEvent.WM_KEYDOWN && vkcode == (int)Keys.R && state.WinPressed)
                 {
                     _winRStroked = true;
                     OnWinRPressed();
                     return false;
                 }
                 if (keyevent == (int)KeyEvent.WM_KEYUP && _winRStroked && vkcode == (int)Keys.LWin)
                 {
                     _winRStroked = false;
                     _keyboardSimulator.ModifiedKeyStroke(VirtualKeyCode.LWIN, VirtualKeyCode.CONTROL);

Plugins/Flow.Launcher.Plugin.Shell/Settings.cs

@@ -1,24 +1,121 @@
 using System.Collections.Generic;
+using Flow.Launcher.Localization.Attributes;
 
 namespace Flow.Launcher.Plugin.Shell
 {
-    public class Settings
+    public class Settings : BaseModel
     {
-        public Shell Shell { get; set; } = Shell.Cmd;
+        private Shell _shell = Shell.Cmd;
+        public Shell Shell
+        {
+            get => _shell;
+            set
+            {
+                if (_shell != value)
+                {
+                    _shell = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool ReplaceWinR { get; set; } = false;
+        private bool _replaceWinR = false;
+        public bool ReplaceWinR
+        {
+            get => _replaceWinR;
+            set
+            {
+                if (_replaceWinR != value)
+                {
+                    _replaceWinR = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool CloseShellAfterPress { get; set; } = false;
+        private bool _closeShellAfterPress = false;
+        public bool CloseShellAfterPress
+        {
+            get => _closeShellAfterPress;
+            set
+            {
+                if (_closeShellAfterPress != value)
+                {
+                    _closeShellAfterPress = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool LeaveShellOpen { get; set; }
+        private bool _leaveShellOpen;
+        public bool LeaveShellOpen
+        {
+            get => _leaveShellOpen;
+            set
+            {
+                if (_leaveShellOpen != value)
+                {
+                    _leaveShellOpen = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool RunAsAdministrator { get; set; } = true;
+        private bool _runAsAdministrator = true;
+        public bool RunAsAdministrator
+        {
+            get => _runAsAdministrator;
+            set
+            {
+                if (_runAsAdministrator != value)
+                {
+                    _runAsAdministrator = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool UseWindowsTerminal { get; set; } = false;
+        private bool _useWindowsTerminal = false;
+        public bool UseWindowsTerminal
+        {
+            get => _useWindowsTerminal;
+            set
+            {
+                if (_useWindowsTerminal != value)
+                {
+                    _useWindowsTerminal = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public bool ShowOnlyMostUsedCMDs { get; set; }
+        private bool _showOnlyMostUsedCMDs;
+        public bool ShowOnlyMostUsedCMDs
+        {
+            get => _showOnlyMostUsedCMDs;
+            set
+            {
+                if (_showOnlyMostUsedCMDs != value)
+                {
+                    _showOnlyMostUsedCMDs = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
-        public int ShowOnlyMostUsedCMDsNumber { get; set; }
+        private int _showOnlyMostUsedCMDsNumber;
+        public int ShowOnlyMostUsedCMDsNumber
+        {
+            get => _showOnlyMostUsedCMDsNumber;
+            set
+            {
+                if (_showOnlyMostUsedCMDsNumber != value)
+                {
+                    _showOnlyMostUsedCMDsNumber = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
 
         public Dictionary<string, int> CommandHistory { get; set; } = [];
 
@@ -31,11 +128,19 @@ public void AddCmdHistory(string cmdName)
         }
     }
 
+    [EnumLocalize]
     public enum Shell
     {
+        [EnumLocalizeValue("CMD")]
         Cmd = 0,
+
+        [EnumLocalizeValue("PowerShell")]
         Powershell = 1,
+
+        [EnumLocalizeValue("RunCommand")]
         RunCommand = 2,
+
+        [EnumLocalizeValue("Pwsh")]
         Pwsh = 3,
     }
 }