bring back alternate row colors for uno

Author: w-ahmad

src/TableView.Properties.cs

@@ -125,7 +125,6 @@ public partial class TableView
     /// </summary>
     public static readonly DependencyProperty VerticalGridLinesStrokeProperty = DependencyProperty.Register(nameof(VerticalGridLinesStroke), typeof(Brush), typeof(TableView), new PropertyMetadata(default, OnGridLinesPropertyChanged));
 
-#if WINDOWS
     /// <summary>
     /// Identifies the AlternateRowForeground dependency property.
     /// </summary>
@@ -136,6 +135,7 @@ public partial class TableView
     /// </summary>
     public static readonly DependencyProperty AlternateRowBackgroundProperty = DependencyProperty.Register(nameof(AlternateRowBackground), typeof(Brush), typeof(TableView), new PropertyMetadata(null, OnAlternateRowColorChanged));
 
+#if WINDOWS
     /// <summary>
     /// Identifies the RowContextFlyout dependency property.
     /// </summary>
@@ -431,7 +431,6 @@ public Brush HorizontalGridLinesStroke
         set => SetValue(HorizontalGridLinesStrokeProperty, value);
     }
 
-#if WINDOWS
     /// <summary>
     /// Gets or sets the background brush for alternate rows.
     /// </summary>
@@ -450,6 +449,7 @@ public Brush AlternateRowForeground
         set => SetValue(AlternateRowForegroundProperty, value);
     }
 
+#if WINDOWS
     /// <summary>
     /// Gets or sets the context flyout for rows.
     /// </summary>

src/TableView.cs

@@ -1303,15 +1303,13 @@ private void EnsureGridLines()
     /// </summary>
     internal void EnsureAlternateRowColors()
     {
-#if WINDOWS
         DispatcherQueue.TryEnqueue(() =>
         {
             foreach (var row in _rows)
             {
                 row.EnsureAlternateColors();
             }
         }); 
-#endif
     }
 
     /// <summary>

src/TableViewRow.cs

@@ -26,11 +26,9 @@ public partial class TableViewRow : ListViewItem
     private ListViewItemPresenter? _itemPresenter;
     private TableViewCellsPresenter? _cellPresenter;
     private Border? _selectionBackground;
-#if WINDOWS
     private bool _ensureCells = true;
     private Brush? _cellPresenterBackground;
-    private Brush? _cellPresenterForeground; 
-#endif
+    private Brush? _cellPresenterForeground;
 
     /// <summary>
     /// Initializes a new instance of the TableViewRow class.
@@ -121,21 +119,17 @@ protected override void OnIsSelectedChanged()
     /// </summary>
     private void OnForegroundChanged()
     {
-#if WINDOWS
         _cellPresenterForeground = Foreground;
-        EnsureAlternateColors(); 
-#endif
+        EnsureAlternateColors();
     }
 
     /// <summary>
     /// Handles the Background property changed.
     /// </summary>
     private void OnBackgroundChanged()
     {
-#if WINDOWS
         _cellPresenterBackground = Background;
-        EnsureAlternateColors(); 
-#endif
+        EnsureAlternateColors();
     }
 
     /// <summary>
@@ -153,10 +147,8 @@ protected override void OnApplyTemplate()
     {
         base.OnApplyTemplate();
 
-#if WINDOWS
         _cellPresenterBackground = Background;
-        _cellPresenterForeground = Foreground; 
-#endif
+        _cellPresenterForeground = Foreground;
         _itemPresenter = GetTemplateChild("Root") as ListViewItemPresenter;
 
         if (_itemPresenter is not null)
@@ -184,10 +176,10 @@ protected override void OnContentChanged(object oldContent, object newContent)
         else
         {
 #endif
-        foreach (var cell in Cells)
-        {
-            cell.RefreshElement();
-        }
+            foreach (var cell in Cells)
+            {
+                cell.RefreshElement();
+            }
 #if WINDOWS
         }
 #endif
@@ -243,8 +235,8 @@ internal void EnsureCells()
             CellPresenter.Children.Clear();
 
             AddCells(TableView.Columns.VisibleColumns);
-        _ensureCells = false;
-    } 
+            _ensureCells = false;
+        }
     }
 
     /// <summary>
@@ -516,7 +508,6 @@ internal void EnsureLayout()
         }
     }
 
-#if WINDOWS
     /// <summary>
     /// Ensures alternate colors are applied to the row.
     /// </summary>
@@ -529,8 +520,7 @@ internal void EnsureAlternateColors()
 
         CellPresenter.Foreground =
             Index % 2 == 1 && TableView.AlternateRowForeground is not null ? TableView.AlternateRowForeground : _cellPresenterForeground;
-    } 
-#endif
+    }
 
     /// <summary>
     /// Gets the list of cells in the row.