feat: improve ui

Author: paul-fresquet

src/ByteSync.Client/Business/Comparisons/ValidationFailureSummary.cs

@@ -5,10 +5,13 @@ namespace ByteSync.Business.Comparisons;
 public class ValidationFailureSummary
 {
     public AtomicActionValidationFailureReason Reason { get; set; }
+    
     public int Count { get; set; }
+    
     public string LocalizedMessage { get; set; } = string.Empty;
+    
     public List<ComparisonItem> AffectedItems { get; set; } = new();
 
     public string AffectedItemsTooltip => string.Join("\n", AffectedItems.Select(item => 
-        item.PathIdentity.FileName ?? item.PathIdentity.LinkingKeyValue));
+        item.PathIdentity.LinkingKeyValue));
 }

src/ByteSync.Client/ViewModels/Sessions/Comparisons/Actions/TargetedActionGlobalViewModel.cs

@@ -128,8 +128,7 @@ public TargetedActionGlobalViewModel(List<ComparisonItem> comparisonItems,
     [Reactive]
     public bool IsInconsistentWithNoValidItems { get; set; }
 
-    [Reactive] 
-    public List<ValidationFailureSummary> FailureSummaries { get; set; } = new();
+    public ObservableCollection<ValidationFailureSummary> FailureSummaries { get; set; } = new();
 
     [Reactive] 
     public string ActionIssuesHeaderMessage { get; set; } = string.Empty;
@@ -287,6 +286,7 @@ private void ShowMissingFieldsWarning()
         AreMissingFields = true;
         IsInconsistentWithValidItems = null;
         IsInconsistentWithNoValidItems = false;
+        FailureSummaries.Clear(); // Clear previous validation failure summaries
 
         DoShowWarning();
     }
@@ -308,7 +308,8 @@ private void ShowConsistencyWarning(AtomicActionConsistencyCheckCanAddResult che
         }
 
         // Generate failure summaries with detailed information
-        FailureSummaries = checkCanAddResult.FailedValidations
+        FailureSummaries.Clear();
+        var summaries = checkCanAddResult.FailedValidations
             .GroupBy(f => f.FailureReason!.Value)
             .Select(g => new ValidationFailureSummary 
             {
@@ -317,8 +318,12 @@ private void ShowConsistencyWarning(AtomicActionConsistencyCheckCanAddResult che
                 LocalizedMessage = _failureReasonLocalizer.GetLocalizedMessage(g.Key),
                 AffectedItems = g.Select(f => f.ComparisonItem).ToList()
             })
-            .OrderByDescending(s => s.Count) // Most frequent failures first
-            .ToList();
+            .OrderByDescending(s => s.Count); // Most frequent failures first
+        
+        foreach (var summary in summaries)
+        {
+            FailureSummaries.Add(summary);
+        }
 
         AreMissingFields = false;