update to WebSearch's custom icon image save behaviour

Load icon image preview on select and only save changes on confirm

Author: jjw24

Plugins/Flow.Launcher.Plugin.WebSearch/SearchSource.cs

@@ -34,11 +34,6 @@ public string IconPath
             }
         }
 
-        [JsonIgnore]
-        public ImageSource Image => ImageLoader.Load(IconPath);
-
-        internal void NotifyImageChange() => OnPropertyChanged(nameof(Image));
-
         public string Url { get; set; }
         public bool Enabled { get; set; }
 

Plugins/Flow.Launcher.Plugin.WebSearch/SearchSourceSetting.xaml

@@ -47,7 +47,7 @@
         <TextBlock Margin="10" FontSize="14" Grid.Row="4" Grid.Column="0" VerticalAlignment="Center"
                    HorizontalAlignment="Right" Text="{DynamicResource flowlauncher_plugin_websearch_icon}" />
         <StackPanel Orientation="Horizontal" Grid.Row="4" Grid.Column="1" Margin="10">
-            <Image Source="{Binding SearchSource.Image ,IsAsync=True}" Width="24" Height="24" Margin="0 0 20 0" />
+            <Image Name="imgPreviewIcon" Width="24" Height="24" Margin="0 0 20 0" />
             <Button Click="OnSelectIconClick" Height="35"
                     Content="{DynamicResource flowlauncher_plugin_websearch_select_icon}" />
         </StackPanel>

Plugins/Flow.Launcher.Plugin.WebSearch/SearchSourceSetting.xaml.cs

@@ -14,6 +14,7 @@ public partial class SearchSourceSettingWindow
         private PluginInitContext _context;
         private IPublicAPI _api;
         private SearchSourceViewModel _viewModel;
+        private string selectedNewIconImageFullPath;
 
 
         public SearchSourceSettingWindow(IList<SearchSource> sources, PluginInitContext context, SearchSource old)
@@ -40,6 +41,8 @@ private void Initilize(IList<SearchSource> sources, PluginInitContext context, A
             _api = _context.API;
 
             _viewModel.SetupCustomImagesDirectory();
+
+            imgPreviewIcon.Source = _viewModel.LoadPreviewIcon(_searchSource.IconPath);
         }
 
         private void OnCancelButtonClick(object sender, RoutedEventArgs e)
@@ -112,6 +115,14 @@ private void EditSearchSource()
                 var warning = _api.GetTranslation("newActionKeywordsHasBeenAssigned");
                 MessageBox.Show(warning);
             }
+
+            if (!string.IsNullOrEmpty(selectedNewIconImageFullPath))
+            {
+                _viewModel.UpdateIconAttributes(_searchSource, selectedNewIconImageFullPath);
+
+                _viewModel.CopyNewImageToUserDataDirectoryIfRequired(
+                                   _searchSource, selectedNewIconImageFullPath, _oldSearchSource.IconPath);
+            }
         }
 
         private void OnSelectIconClick(object sender, RoutedEventArgs e)
@@ -122,17 +133,14 @@ private void OnSelectIconClick(object sender, RoutedEventArgs e)
             var result = dialog.ShowDialog();
             if (result == true)
             {
-                var fullpathToSelectedImage = dialog.FileName;
-
-                if (_viewModel.ShouldProvideHint(fullpathToSelectedImage))
-                    MessageBox.Show(_api.GetTranslation("flowlauncher_plugin_websearch_iconpath_hint"));
+                selectedNewIconImageFullPath = dialog.FileName;
 
-                if (!string.IsNullOrEmpty(fullpathToSelectedImage))
+                if (!string.IsNullOrEmpty(selectedNewIconImageFullPath))
                 {
-                        var fullPathToOriginalImage = _searchSource.IconPath;
-                        _viewModel.UpdateIconAttributes(_searchSource, fullpathToSelectedImage);
-                        _viewModel.CopyNewImageToUserDataDirectoryIfRequired(
-                                    _searchSource, fullpathToSelectedImage, fullPathToOriginalImage);
+                    if (_viewModel.ShouldProvideHint(selectedNewIconImageFullPath))
+                        MessageBox.Show(_api.GetTranslation("flowlauncher_plugin_websearch_iconpath_hint"));
+                    
+                    imgPreviewIcon.Source = _viewModel.LoadPreviewIcon(selectedNewIconImageFullPath);
                 }
             }
         }

Plugins/Flow.Launcher.Plugin.WebSearch/SearchSourceViewModel.cs

@@ -1,5 +1,8 @@
+using Flow.Launcher.Infrastructure.Image;
 using System;
+using System.Drawing;
 using System.IO;
+using System.Windows.Media;
 
 namespace Flow.Launcher.Plugin.WebSearch
 {
@@ -40,8 +43,6 @@ public void CopyNewImageToUserDataDirectoryIfRequired(
 #endif
                 }
             }
-
-            selectedSearchSource.NotifyImageChange();
         }
 
         internal void SetupCustomImagesDirectory()
@@ -54,5 +55,10 @@ internal bool ShouldProvideHint(string fullPathToSelectedImage)
         {
             return Directory.GetParent(fullPathToSelectedImage).ToString() == Main.DefaultImagesDirectory;
         }
+
+        internal ImageSource LoadPreviewIcon(string pathToPreviewIconImage)
+        {
+            return ImageLoader.Load(pathToPreviewIconImage);
+        }
     }
 }