Enhancement: Rename Quick Access Links

Plugins/Flow.Launcher.Plugin.Explorer/ContextMenu.cs

@@ -70,16 +70,18 @@
                     contextMenus.Add(new Result
                     {
                         Title = Context.API.GetTranslation("plugin_explorer_add_to_quickaccess_title"),
                         SubTitle = Context.API.GetTranslation("plugin_explorer_add_to_quickaccess_subtitle"),
                         Action = (context) =>
                         {
                             Settings.QuickAccessLinks.Add(new AccessLink
                             {
-                                Path = record.FullPath, Type = record.Type
+                                Name = record.FullPath.GetPathName(),
+                                Path = record.FullPath,
+                                Type = record.Type
                             });
 
                             Context.API.ShowMsg(Context.API.GetTranslation("plugin_explorer_addfilefoldersuccess"),
                                     Context.API.GetTranslation("plugin_explorer_addfilefoldersuccess_detail"),
                                     Constants.ExplorerIconImageFullPath);
 
                             ViewModel.Save();
@@ -87,7 +89,7 @@
                             return true;
                         },
                         SubTitleToolTip = Context.API.GetTranslation("plugin_explorer_contextmenu_titletooltip"),
                         TitleToolTip = Context.API.GetTranslation("plugin_explorer_contextmenu_titletooltip"),
                         IcoPath = Constants.QuickAccessImagePath,
                         Glyph = new GlyphInfo(FontFamily: "/Resources/#Segoe Fluent Icons", Glyph: "\ue718"),
                     });
@@ -96,22 +98,22 @@
                 {
                     contextMenus.Add(new Result
                     {
                         Title = Context.API.GetTranslation("plugin_explorer_remove_from_quickaccess_title"),
                         SubTitle = Context.API.GetTranslation("plugin_explorer_remove_from_quickaccess_subtitle"),
                         Action = (context) =>
                         {
                             Settings.QuickAccessLinks.Remove(Settings.QuickAccessLinks.FirstOrDefault(x => string.Equals(x.Path, record.FullPath, StringComparison.OrdinalIgnoreCase)));
 
                             Context.API.ShowMsg(Context.API.GetTranslation("plugin_explorer_removefilefoldersuccess"),
                                     Context.API.GetTranslation("plugin_explorer_removefilefoldersuccess_detail"),
                                     Constants.ExplorerIconImageFullPath);
 
                             ViewModel.Save();
 
                             return true;
                         },
                         SubTitleToolTip = Context.API.GetTranslation("plugin_explorer_contextmenu_remove_titletooltip"),
                         TitleToolTip = Context.API.GetTranslation("plugin_explorer_contextmenu_remove_titletooltip"),
                         IcoPath = Constants.RemoveQuickAccessImagePath,
                         Glyph = new GlyphInfo(FontFamily: "/Resources/#Segoe Fluent Icons", Glyph: "\uecc9")
                     });

Plugins/Flow.Launcher.Plugin.Explorer/Helper/PathHelper.cs

@@ -0,0 +1,21 @@
+using System;
+using System.IO;
+using System.Linq;
+using Flow.Launcher.Plugin.Explorer.Search;
+
+namespace Flow.Launcher.Plugin.Explorer.Helper;
+
+public static class PathHelper
+{
+    public static string GetPathName(this string selectedPath)
+    {
+        if (string.IsNullOrEmpty(selectedPath)) return string.Empty;
+        var path = selectedPath.EndsWith(Constants.DirectorySeparator) ? selectedPath[0..^1] : selectedPath;
+
+        if (path.EndsWith(':'))
+            return path[0..^1] + " Drive";
+
+        return path.Split(new[] { Path.DirectorySeparatorChar }, StringSplitOptions.None)
+            .Last();
+    }
+}

Plugins/Flow.Launcher.Plugin.Explorer/Languages/en.xaml

@@ -5,6 +5,8 @@
 
     <!--  Dialogues  -->
     <system:String x:Key="plugin_explorer_make_selection_warning">Please make a selection first</system:String>
+    <system:String x:Key="plugin_explorer_quick_access_link_no_folder_selected">Please select a folder path.</system:String>
+    <system:String x:Key="plugin_explorer_quick_access_link_path_already_exists">Please choose a different name or folder path.</system:String>
     <system:String x:Key="plugin_explorer_select_folder_link_warning">Please select a folder link</system:String>
     <system:String x:Key="plugin_explorer_delete_folder_link">Are you sure you want to delete {0}?</system:String>
     <system:String x:Key="plugin_explorer_deletefileconfirm">Are you sure you want to permanently delete this file?</system:String>
@@ -27,6 +29,7 @@
     <system:String x:Key="plugin_explorer_add">Add</system:String>
     <system:String x:Key="plugin_explorer_generalsetting_header">General Setting</system:String>
     <system:String x:Key="plugin_explorer_manageactionkeywords_header">Customise Action Keywords</system:String>
+    <system:String x:Key="plugin_explorer_manage_quick_access_links_header">Customise Quick Access</system:String>
     <system:String x:Key="plugin_explorer_quickaccesslinks_header">Quick Access Links</system:String>
     <system:String x:Key="plugin_explorer_everything_setting_header">Everything Setting</system:String>
     <system:String x:Key="plugin_explorer_previewpanel_setting_header">Preview Panel</system:String>
@@ -92,6 +95,7 @@
     <system:String x:Key="plugin_explorer_deletefile_subtitle">Permanently delete current file</system:String>
     <system:String x:Key="plugin_explorer_deletefolder_subtitle">Permanently delete current folder</system:String>
     <system:String x:Key="plugin_explorer_path">Path:</system:String>
+    <system:String x:Key="plugin_explorer_name">Name:</system:String>
     <system:String x:Key="plugin_explorer_deletefilefolder_subtitle">Delete the selected</system:String>
     <system:String x:Key="plugin_explorer_runasdifferentuser">Run as different user</system:String>
     <system:String x:Key="plugin_explorer_runasdifferentuser_subtitle">Run the selected using a different user account</system:String>

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

@@ -35,6 +35,7 @@ public Task InitAsync(PluginInitContext context)
             Context = context;
 
             Settings = context.API.LoadSettingJsonStorage<Settings>();
+            FillQuickAccessLinkNames();
 
             viewModel = new SettingsViewModel(context, Settings);
 
@@ -95,5 +96,17 @@ public string GetTranslatedPluginDescription()
         {
             return Context.API.GetTranslation("plugin_explorer_plugin_description");
         }
+
+        private void FillQuickAccessLinkNames()
+        {
+            // Legacy version does not have names for quick access links, so we fill them with the path name.
+            foreach (var link in Settings.QuickAccessLinks)
+            {
+                if (string.IsNullOrWhiteSpace(link.Name))
+                {
+                    link.Name = link.Path.GetPathName();
+                }
+            }
+        }
     }
 }

Plugins/Flow.Launcher.Plugin.Explorer/Search/QuickAccessLinks/AccessLink.cs

@@ -1,29 +1,11 @@
-using System;
-using System.Linq;
-using System.Text.Json.Serialization;
-
-namespace Flow.Launcher.Plugin.Explorer.Search.QuickAccessLinks
+namespace Flow.Launcher.Plugin.Explorer.Search.QuickAccessLinks
 {
     public class AccessLink
     {
         public string Path { get; set; }
 
         public ResultType Type { get; set; } = ResultType.Folder;
 
-        [JsonIgnore]
-        public string Name
-        {
-            get
-            {
-                var path = Path.EndsWith(Constants.DirectorySeparator) ? Path[0..^1] : Path;
-
-                if (path.EndsWith(':'))
-                    return path[0..^1] + " Drive";
-
-                return path.Split(new[] { System.IO.Path.DirectorySeparatorChar }, StringSplitOptions.None)
-                           .Last();
-            }
-        }
+        public string Name { get; set; }
     }
-
 }

Plugins/Flow.Launcher.Plugin.Explorer/ViewModels/SettingsViewModel.cs

@@ -9,6 +9,7 @@
 using System.Windows;
 using System.Windows.Forms;
 using CommunityToolkit.Mvvm.Input;
+using Flow.Launcher.Plugin.Explorer.Helper;
 using Flow.Launcher.Plugin.Explorer.Search;
 using Flow.Launcher.Plugin.Explorer.Search.Everything;
 using Flow.Launcher.Plugin.Explorer.Search.Everything.Exceptions;
@@ -328,14 +329,10 @@ public void AppendLink(string containerName, AccessLink link)
         }
 
         [RelayCommand]
-        private void EditLink(object commandParameter)
+        private void EditIndexSearchExcludePaths()
         {
-            var (selectedLink, collection) = commandParameter switch
-            {
-                "QuickAccessLink" => (SelectedQuickAccessLink, Settings.QuickAccessLinks),
-                "IndexSearchExcludedPaths" => (SelectedIndexSearchExcludedPath, Settings.IndexSearchExcludedSubdirectoryPaths),
-                _ => throw new ArgumentOutOfRangeException(nameof(commandParameter))
-            };
+            var selectedLink = SelectedIndexSearchExcludedPath;
+            var collection = Settings.IndexSearchExcludedSubdirectoryPaths;
 
             if (selectedLink is null)
             {
@@ -354,45 +351,66 @@ private void EditLink(object commandParameter)
             collection.Remove(selectedLink);
             collection.Add(new AccessLink
             {
-                Path = path, Type = selectedLink.Type,
+                Path = path, Type = selectedLink.Type, Name = path.GetPathName()
             });
-        }
-
-        private void ShowUnselectedMessage()
-        {
-            var warning = Context.API.GetTranslation("plugin_explorer_make_selection_warning");
-            Context.API.ShowMsgBox(warning);
+            Save();
         }
 
         [RelayCommand]
-        private void AddLink(object commandParameter)
+        private void AddIndexSearchExcludePaths()
         {
-            var container = commandParameter switch
-            {
-                "QuickAccessLink" => Settings.QuickAccessLinks,
-                "IndexSearchExcludedPaths" => Settings.IndexSearchExcludedSubdirectoryPaths,
-                _ => throw new ArgumentOutOfRangeException(nameof(commandParameter))
-            };
-
-            ArgumentNullException.ThrowIfNull(container);
+            var container = Settings.IndexSearchExcludedSubdirectoryPaths;
 
+            if (container is null) return;
+
             var folderBrowserDialog = new FolderBrowserDialog();
 
             if (folderBrowserDialog.ShowDialog() != DialogResult.OK)
                 return;
 
             var newAccessLink = new AccessLink
             {
+                Name = folderBrowserDialog.SelectedPath.GetPathName(),
                 Path = folderBrowserDialog.SelectedPath
             };
 
             container.Add(newAccessLink);
+            Save();
+        }
+
+        [RelayCommand]
+        private void EditQuickAccessLink()
+        {
+            var selectedLink = SelectedQuickAccessLink;
+            var collection = Settings.QuickAccessLinks;
+
+            if (selectedLink is null)
+            {
+                ShowUnselectedMessage();
+                return;
+            }
+
+            var quickAccessLinkSettings = new QuickAccessLinkSettings(collection, SelectedQuickAccessLink);
+            if (quickAccessLinkSettings.ShowDialog() == true)
+            {
+                Save();
+            }
+        }
+
+        [RelayCommand]
+        private void AddQuickAccessLink()
+        {
+            var quickAccessLinkSettings = new QuickAccessLinkSettings(Settings.QuickAccessLinks);
+            if (quickAccessLinkSettings.ShowDialog() == true)
+            {
+                Save();
+            }
         }
 
         [RelayCommand]
-        private void RemoveLink(object obj)
+        private void RemoveLink(object commandParameter)
         {
-            if (obj is not string container) return;
+            if (commandParameter is not string container) return;
 
             switch (container)
             {
@@ -407,10 +425,16 @@ private void RemoveLink(object obj)
             }
             Save();
         }
+
+        private void ShowUnselectedMessage()
+        {
+            var warning = Context.API.GetTranslation("plugin_explorer_make_selection_warning");
+            Context.API.ShowMsgBox(warning);
+        }
 
         #endregion
 
-        private string? PromptUserSelectPath(ResultType type, string? initialDirectory = null)
+        private static string? PromptUserSelectPath(ResultType type, string? initialDirectory = null)
         {
             string? path = null;