fix bug

Author: LoneWandererProductions

CommonControls/Thumbnails.xaml CommonControls/ThumbnailsNew.xamlCommonControls/Thumbnails.xaml renamed to CommonControls/ThumbnailsNew.xaml

@@ -1,4 +1,4 @@
-<UserControl x:Class="CommonControls.Thumbnails"
+<UserControl x:Class="CommonControls.ThumbnailsNew"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:d="http://schemas.microsoft.com/expression/blend/2008"

CommonControls/Thumbnails.xaml.cs CommonControls/ThumbnailsNew.xaml.csCommonControls/Thumbnails.xaml.cs renamed to CommonControls/ThumbnailsNew.xaml.cs

@@ -10,21 +10,21 @@
 
 namespace CommonControls
 {
-    public partial class Thumbnails : UserControl
+    public partial class ThumbnailsNew : UserControl
     {
         // ObservableCollection automatically notifies the UI when items are added/removed
         public ObservableCollection<ThumbnailItem> ThumbCollection { get; set; } = new();
 
         public static readonly DependencyProperty ThumbCellSizeProperty =
-            DependencyProperty.Register(nameof(ThumbCellSize), typeof(double), typeof(Thumbnails), new PropertyMetadata(100.0));
+            DependencyProperty.Register(nameof(ThumbCellSize), typeof(double), typeof(ThumbnailsNew), new PropertyMetadata(100.0));
 
         public double ThumbCellSize
         {
             get => (double)GetValue(ThumbCellSizeProperty);
             set => SetValue(ThumbCellSizeProperty, value);
         }
 
-        public Thumbnails()
+        public ThumbnailsNew()
         {
             InitializeComponent();
             // Set the DataContext to this control so XAML can bind to ThumbCollection
@@ -36,36 +36,34 @@ public async Task LoadImages(Dictionary<int, string> sourceList)
             ThumbCollection.Clear();
 
             // Using your optimized loading logic, but slightly cleaner
-            using (var semaphore = new SemaphoreSlim(4))
+            using var semaphore = new SemaphoreSlim(4);
+            var tasks = sourceList.Select(async kvp =>
             {
-                var tasks = sourceList.Select(async kvp =>
+                await semaphore.WaitAsync();
+                try
                 {
-                    await semaphore.WaitAsync();
-                    try
+                    var img = await LoadImageAsync(kvp.Value);
+                    if (img != null)
                     {
-                        var img = await LoadImageAsync(kvp.Value);
-                        if (img != null)
+                        // Create the Data Item
+                        var item = new ThumbnailItem
                         {
-                            // Create the Data Item
-                            var item = new ThumbnailItem
-                            {
-                                Id = kvp.Key,
-                                FilePath = kvp.Value,
-                                ImageSource = img
-                            };
+                            Id = kvp.Key,
+                            FilePath = kvp.Value,
+                            ImageSource = img
+                        };
 
-                            // Thread-safe add to collection
-                            Application.Current.Dispatcher.Invoke(() => ThumbCollection.Add(item));
-                        }
-                    }
-                    finally
-                    {
-                        semaphore.Release();
+                        // Thread-safe add to collection
+                        Application.Current.Dispatcher.Invoke(() => ThumbCollection.Add(item));
                     }
-                });
+                }
+                finally
+                {
+                    semaphore.Release();
+                }
+            });
 
-                await Task.WhenAll(tasks);
-            }
+            await Task.WhenAll(tasks);
         }
 
         private async Task<BitmapSource> LoadImageAsync(string path)
@@ -76,15 +74,13 @@ private async Task<BitmapSource> LoadImageAsync(string path)
                 {
                     if (!File.Exists(path)) return null;
                     var bi = new BitmapImage();
-                    using (var fs = new FileStream(path, FileMode.Open, FileAccess.Read))
-                    {
-                        bi.BeginInit();
-                        bi.StreamSource = fs;
-                        bi.CacheOption = BitmapCacheOption.OnLoad;
-                        bi.DecodePixelWidth = (int)ThumbCellSize; // Decode small to save RAM
-                        bi.EndInit();
-                        bi.Freeze(); // Vital for cross-thread
-                    }
+                    using var fs = new FileStream(path, FileMode.Open, FileAccess.Read);
+                    bi.BeginInit();
+                    bi.StreamSource = fs;
+                    bi.CacheOption = BitmapCacheOption.OnLoad;
+                    bi.DecodePixelWidth = (int)ThumbCellSize; // Decode small to save RAM
+                    bi.EndInit();
+                    bi.Freeze(); // Vital for cross-thread
                     return bi;
                 }
                 catch { return null; }