Fix unselected issue for bookmark plugin by using binding (Don't understand Reason it doesn't work before)

Author: taooceros

Plugins/Flow.Launcher.Plugin.BrowserBookmark/Views/SettingsControl.xaml

@@ -22,12 +22,12 @@
                 </Grid.ColumnDefinitions>
                 <Label Grid.Column="0" Content="{DynamicResource flowlauncher_plugin_browserbookmark_settings_openBookmarks}"
                        FontSize="15" Margin="10 5 0 0"/>
-                <RadioButton Grid.Column="1" Name="NewWindowBrowser" GroupName="OpenSearchBehaviour" 
-                             Content="{DynamicResource flowlauncher_plugin_browserbookmark_settings_newWindow}"
-                             Click="OnNewBrowserWindowClick" />
-                <RadioButton Grid.Column="2" Name="NewTabInBrowser" GroupName="OpenSearchBehaviour" 
-                             Content="{DynamicResource flowlauncher_plugin_browserbookmark_settings_newTab}" 
-                             Click="OnNewTabClick" />
+                <RadioButton Grid.Column="1" Name="NewWindowBrowser"
+                             IsChecked="{Binding OpenInNewBrowserWindow}"
+                             Content="{DynamicResource flowlauncher_plugin_browserbookmark_settings_newWindow}"/>
+                <RadioButton Grid.Column="2" Name="NewTabInBrowser"
+                             IsChecked="{Binding OpenInNewTab, Mode=OneTime}"
+                             Content="{DynamicResource flowlauncher_plugin_browserbookmark_settings_newTab}"/>
             </Grid>
         </StackPanel>
         <StackPanel Orientation="Horizontal" VerticalAlignment="Top" Grid.Row="1" Height="60" Margin="30,20,0,0">

Plugins/Flow.Launcher.Plugin.BrowserBookmark/Views/SettingsControl.xaml.cs

@@ -3,35 +3,39 @@
 using System.Windows.Controls;
 using Flow.Launcher.Plugin.BrowserBookmark.Models;
 using System.Windows.Input;
+using System.ComponentModel;
 
 namespace Flow.Launcher.Plugin.BrowserBookmark.Views
 {
     /// <summary>
     /// Interaction logic for BrowserBookmark.xaml
     /// </summary>
-    public partial class SettingsControl
+    public partial class SettingsControl : INotifyPropertyChanged
     {
         public Settings Settings { get; }
         public CustomBrowser SelectedCustomBrowser { get; set; }
-
-        public SettingsControl(Settings settings)
+        public bool OpenInNewBrowserWindow
         {
-            Settings = settings;
-            InitializeComponent();
-            NewWindowBrowser.IsChecked = Settings.OpenInNewBrowserWindow;
-            NewTabInBrowser.IsChecked = !Settings.OpenInNewBrowserWindow;
+            get => Settings.OpenInNewBrowserWindow;
+            set
+            {
+                Settings.OpenInNewBrowserWindow = value;
+                PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(nameof(OpenInNewBrowserWindow)));
+            }
         }
-
-        private void OnNewBrowserWindowClick(object sender, RoutedEventArgs e)
+        public bool OpenInNewTab
         {
-            Settings.OpenInNewBrowserWindow = true;
+            get => !OpenInNewBrowserWindow;
         }
 
-        private void OnNewTabClick(object sender, RoutedEventArgs e)
+        public SettingsControl(Settings settings)
         {
-            Settings.OpenInNewBrowserWindow = false;
+            Settings = settings;
+            InitializeComponent();
         }
 
+        public event PropertyChangedEventHandler PropertyChanged;
+
         private void OnChooseClick(object sender, RoutedEventArgs e)
         {
             var fileBrowserDialog = new OpenFileDialog();
@@ -61,7 +65,7 @@ private void NewCustomBrowser(object sender, RoutedEventArgs e)
 
         private void DeleteCustomBrowser(object sender, RoutedEventArgs e)
         {
-            if(CustomBrowsers.SelectedItem is CustomBrowser selectedCustomBrowser)
+            if (CustomBrowsers.SelectedItem is CustomBrowser selectedCustomBrowser)
             {
                 Settings.CustomChromiumBrowsers.Remove(selectedCustomBrowser);
             }