fixed selection check mark states

Author: w-ahmad

src/TableView.Properties.cs

@@ -209,7 +209,7 @@ public partial class TableView
     /// <summary>
     /// Gets or sets a value indicating whether the TableView is in editing mode.
     /// </summary>
-    internal bool IsEditing { get; set; }
+    internal bool IsEditing { get; private set; }
 
     /// <summary>
     /// Gets the width of the selection indicator.

src/TableView.cs

@@ -154,8 +154,8 @@ private void HandleNavigations(KeyRoutedEventArgs e, bool shiftKey, bool ctrlKey
         }
         else if (e.Key is VirtualKey.Escape && currentCell is not null && IsEditing)
         {
+            SetIsEditing(false);
             currentCell?.SetElement();
-            IsEditing = false;
             e.Handled = true;
         }
         else if (e.Key is VirtualKey.Space && currentCell is not null && CurrentCellSlot.HasValue && !IsEditing)
@@ -1392,7 +1392,7 @@ internal void UpdateCornerButtonState()
     {
         _headerRow?.SetCornerButtonState();
 
-        DispatcherQueue.TryEnqueue(Microsoft.UI.Dispatching.DispatcherQueuePriority.Low,() =>
+        DispatcherQueue.TryEnqueue(Microsoft.UI.Dispatching.DispatcherQueuePriority.Low, () =>
         {
             if (SelectionMode is ListViewSelectionMode.Multiple && SelectionUnit is not TableViewSelectionUnit.Cell)
             {
@@ -1404,6 +1404,17 @@ internal void UpdateCornerButtonState()
         });
     }
 
+    internal void SetIsEditing(bool value)
+    {
+        if (IsEditing == value)
+        {
+            return;
+        }
+
+        IsEditing = value;
+        UpdateCornerButtonState();
+    }
+
     public virtual void OnSorting(TableViewSortingEventArgs eventArgs)
     {
         Sorting?.Invoke(this, eventArgs);

src/TableViewCell.cs

@@ -284,10 +284,11 @@ private void MakeSelection()
                 TableView.DeselectCell(Slot);
             }
 
-            TableView.IsEditing = false;
-            TableView.UpdateCornerButtonState();
             TableView.MakeSelection(Slot, shiftKey, ctrlKey);
         }
+
+        TableView.SetIsEditing(false);
+        TableView.UpdateCornerButtonState();
     }
 
     /// <summary>
@@ -339,7 +340,7 @@ private void SetEditingElement()
 
         if (TableView is not null)
         {
-            TableView.IsEditing = true;
+            TableView.SetIsEditing(true);
             TableView.UpdateCornerButtonState();
         }
     }

src/TableViewRow.cs

@@ -19,6 +19,7 @@ public partial class TableViewRow : ListViewItem
 {
     private const string Selection_Indictor = nameof(Selection_Indictor);
     private const string Selection_Background = nameof(Selection_Background);
+    private const string Check_Mark = "\uE73E";
     private Thickness _focusVisualMargin = new(1);
     private Thickness _selectionBackgroundMargin = new(4, 2, 4, 2);
 
@@ -525,7 +526,7 @@ internal void EnsureAlternateColors()
 
     internal void UpdateSelectCheckMarkOpacity()
     {
-        var fontIcon = this.FindDescendant<FontIcon>(x => x.Parent is Border);
+        var fontIcon = this.FindDescendant<FontIcon>(x => x.Glyph == Check_Mark);
 
         if (fontIcon?.Parent is Border border)
         {