Update menu when pinning or unpinning.

src/PicView.Avalonia/UI/FileHistory/FileHistoryMenuItem.cs

@@ -2,153 +2,180 @@
 using Avalonia.Controls;
 using Avalonia.Layout;
 using Avalonia.Media;
+using Avalonia.Threading;
 using PicView.Avalonia.Navigation;
 using PicView.Avalonia.ViewModels;
 using PicView.Avalonia.Views.UC.Buttons;
 using PicView.Core.Extensions;
 using PicView.Core.FileHistory;
 
-namespace PicView.Avalonia.UI.FileHistory;
-
-/// <summary>
-///     Represents a single file history menu item with pin/unpin functionality
-/// </summary>
-public class FileHistoryMenuItem : Panel
+namespace PicView.Avalonia.UI.FileHistory
 {
-    private const int MaxFilenameLength = 42;
-
-    public FileHistoryMenuItem(Entry entry, string? currentFilePath, MainViewModel viewModel, int index)
+    /// <summary>
+    ///     Represents a single file history menu item with pin/unpin functionality
+    /// </summary>
+    public class FileHistoryMenuItem : Panel
     {
-        var fileLocation = entry.Path;
-        if (string.IsNullOrEmpty(fileLocation))
-        {
-            return;
-        }
+        private const int MaxFilenameLength = 42;
 
-        var isSelected = fileLocation == currentFilePath;
-        var filename = Path.GetFileName(fileLocation);
-        var header = filename.Length > MaxFilenameLength ? filename.Shorten(MaxFilenameLength) : filename;
-
-        // Create the pin button with appropriate visibility
-        var pinButton = CreatePinButton(entry, fileLocation);
-
-        // Create the menu item button with file info
-        var menuItemButton = CreateMenuItemButton(header, fileLocation, isSelected, index, viewModel);
+        public FileHistoryMenuItem(Entry entry, string? currentFilePath, MainViewModel viewModel, int index)
+        {
+            var fileLocation = entry.Path;
+            if (string.IsNullOrEmpty(fileLocation))
+            {
+                return;
+            }
 
-        // Add components to the panel
-        Children.Add(menuItemButton);
-        Children.Add(pinButton);
+            var isSelected = fileLocation == currentFilePath;
+            var filename = Path.GetFileName(fileLocation);
+            var header = filename.Length > MaxFilenameLength ? filename.Shorten(MaxFilenameLength) : filename;
 
-        // Add hover behavior
-        ConfigureHoverBehavior(pinButton);
+            // Create the pin button with appropriate visibility
+            var pinButton = CreatePinButton(entry, fileLocation);
 
-        // Set tooltip
-        ToolTip.SetTip(menuItemButton, fileLocation);
-    }
+            // Create the menu item button with file info
+            var menuItemButton = CreateMenuItemButton(header, fileLocation, isSelected, index, viewModel);
 
-    private static PinButton CreatePinButton(Entry entry, string fileLocation)
-    {
-        var pinBtn = new PinButton
-        {
-            Name = "PinBtn",
-            Opacity = 0,
-            Width = 25,
-            HorizontalAlignment = HorizontalAlignment.Right,
-            ZIndex = 1
-        };
+            // Add components to the panel
+            Children.Add(menuItemButton);
+            Children.Add(pinButton);
 
-        pinBtn.PinBtn.Click += (_, _) => FileHistoryManager.Pin(fileLocation);
-        pinBtn.UnPinBtn.Click += (_, _) => FileHistoryManager.UnPin(fileLocation);
+            // Add hover behavior
+            ConfigureHoverBehavior(pinButton);
 
-        if (entry.IsPinned)
-        {
-            pinBtn.PinBtn.IsVisible = false;
-            pinBtn.UnPinBtn.IsVisible = true;
+            // Set tooltip
+            ToolTip.SetTip(menuItemButton, fileLocation);
         }
-        else
-        {
-            pinBtn.PinBtn.IsVisible = true;
-            pinBtn.UnPinBtn.IsVisible = false;
-        }
-
-        return pinBtn;
-    }
-
-    private static Button CreateMenuItemButton(string header, string fileLocation, bool isSelected, int index,
-        MainViewModel viewModel)
-    {
-        var item = new Button
-        {
-            Background = Brushes.Transparent,
-            Padding = new Thickness(5, 6),
-            Width = 355
-        };
 
-        if (index < 0)
+        private static PinButton CreatePinButton(Entry entry, string fileLocation)
         {
-            // Pinned item without index number
-            item.Padding = new Thickness(15, 0, 0, 0);
-            item.Content = new TextBlock
+            var pinBtn = new PinButton
             {
-                Classes = { "txt" },
-                Text = header,
-                Padding = new Thickness(5, 5, 0, 5)
+                Opacity = 0,
+                Width = 25,
+                HorizontalAlignment = HorizontalAlignment.Right,
+                ZIndex = 1
             };
-        }
-        else
-        {
-            // Regular item with index number
-            var indexText = new TextBlock
+
+            // Toggle just this button's visibility instead of rebuilding the whole menu
+            pinBtn.PinBtn.Click += (_, _) => 
             {
-                Classes = { "txt" },
-                Text = (index + 1).ToString(),
-                Padding = new Thickness(5, 0, 2, 0)
+                FileHistoryManager.Pin(fileLocation);
+                // Just update this button's visibility
+                pinBtn.PinBtn.IsVisible = false;
+                pinBtn.UnPinBtn.IsVisible = true;
+        
+                // Schedule a deferred menu refresh to happen after this event has completed
+                Dispatcher.UIThread.Post(() =>
+                {
+                    UIHelper.GetMainView.FileHistoryMenuController.UpdateFileHistoryMenu();
+                });
             };
-
-            var headerText = new TextBlock
+    
+            pinBtn.UnPinBtn.Click += (_, _) => 
             {
-                Classes = { "txt" },
-                Text = header,
-                Padding = new Thickness(5, 0, 0, 0)
+                FileHistoryManager.UnPin(fileLocation);
+                // Just update this button's visibility
+                pinBtn.PinBtn.IsVisible = true;
+                pinBtn.UnPinBtn.IsVisible = false;
+        
+                // Schedule a deferred menu refresh to happen after this event has completed
+                Dispatcher.UIThread.Post(() =>
+                {
+                    UIHelper.GetMainView.FileHistoryMenuController.UpdateFileHistoryMenu();
+                });
             };
 
-            item.Content = new StackPanel
+            if (entry.IsPinned)
             {
-                Orientation = Orientation.Horizontal,
-                Children = { indexText, headerText }
-            };
-        }
+                pinBtn.PinBtn.IsVisible = false;
+                pinBtn.UnPinBtn.IsVisible = true;
+            }
+            else
+            {
+                pinBtn.PinBtn.IsVisible = true;
+                pinBtn.UnPinBtn.IsVisible = false;
+            }
 
-        if (isSelected)
-        {
-            item.Classes.Add("active");
+            return pinBtn;
         }
 
-        item.Click += async delegate
+        private static Button CreateMenuItemButton(string header, string fileLocation, bool isSelected, int index,
+            MainViewModel viewModel)
         {
-            UIHelper.GetMainView.MainContextMenu.Close();
-            await NavigationManager.LoadPicFromStringAsync(fileLocation, viewModel).ConfigureAwait(false);
-        };
+            var item = new Button
+            {
+                Background = Brushes.Transparent,
+                Padding = new Thickness(5, 6),
+                Width = 355
+            };
 
-        return item;
-    }
+            if (index < 0)
+            {
+                // Pinned item without index number
+                item.Padding = new Thickness(15, 0, 0, 0);
+                item.Content = new TextBlock
+                {
+                    Classes = { "txt" },
+                    Text = header,
+                    Padding = new Thickness(5, 5, 0, 5)
+                };
+            }
+            else
+            {
+                // Regular item with index number
+                var indexText = new TextBlock
+                {
+                    Classes = { "txt" },
+                    Text = (index + 1).ToString(),
+                    Padding = new Thickness(5, 0, 2, 0)
+                };
+
+                var headerText = new TextBlock
+                {
+                    Classes = { "txt" },
+                    Text = header,
+                    Padding = new Thickness(5, 0, 0, 0)
+                };
+
+                item.Content = new StackPanel
+                {
+                    Orientation = Orientation.Horizontal,
+                    Children = { indexText, headerText }
+                };
+            }
 
-    private void ConfigureHoverBehavior(PinButton pinBtn)
-    {
-        PointerEntered += (_, _) =>
-        {
-            pinBtn.Opacity = 1;
-            if (Application.Current.TryGetResource("AccentColor", Application.Current.RequestedThemeVariant,
-                    out var accentColor))
+            if (isSelected)
             {
-                Background = accentColor as SolidColorBrush;
+                item.Classes.Add("active");
             }
-        };
 
-        PointerExited += (_, _) =>
+            item.Click += async delegate
+            {
+                UIHelper.GetMainView.MainContextMenu.Close();
+                await NavigationManager.LoadPicFromStringAsync(fileLocation, viewModel).ConfigureAwait(false);
+            };
+
+            return item;
+        }
+
+        private void ConfigureHoverBehavior(PinButton pinBtn)
         {
-            pinBtn.Opacity = 0;
-            Background = Brushes.Transparent;
-        };
+            PointerEntered += (_, _) =>
+            {
+                pinBtn.Opacity = 1;
+                if (Application.Current.TryGetResource("AccentColor", Application.Current.RequestedThemeVariant,
+                        out var accentColor))
+                {
+                    Background = accentColor as SolidColorBrush;
+                }
+            };
+
+            PointerExited += (_, _) =>
+            {
+                pinBtn.Opacity = 0;
+                Background = Brushes.Transparent;
+            };
+        }
     }
 }

src/PicView.Avalonia/Views/MainView.axaml.cs

@@ -18,7 +18,7 @@ namespace PicView.Avalonia.Views;
 
 public partial class MainView : UserControl
 {
-    private FileHistoryMenuController? _historyMenuController;
+    public FileHistoryMenuController? FileHistoryMenuController;
     
     public MainView()
     {
@@ -60,7 +60,7 @@ public partial class MainView : UserControl
                 return;
             }
             // Initialize the history menu controller
-            _historyMenuController = new FileHistoryMenuController(RecentFilesCM, HistorySortButton, HistoryClearButton, vm);
+            FileHistoryMenuController = new FileHistoryMenuController(RecentFilesCM, HistorySortButton, HistoryClearButton, vm);
 
             HideInterfaceLogic.AddHoverButtonEvents(AltButtonsPanel, vm);
             PointerWheelChanged += async (_, e) => await vm.ImageViewer.PreviewOnPointerWheelChanged(this, e);
@@ -127,7 +127,7 @@ public partial class MainView : UserControl
         }, DispatcherPriority.Render);
         
         // Update file history menu items in Dispatcher with low priority to avoid slowdown
-        await Dispatcher.UIThread.InvokeAsync(() => _historyMenuController?.UpdateFileHistoryMenu(), DispatcherPriority.Background);
+        await Dispatcher.UIThread.InvokeAsync(() => FileHistoryMenuController?.UpdateFileHistoryMenu(), DispatcherPriority.Background);
     }
 
     private async Task Drop(object? sender, DragEventArgs e)