code_review: PR #1423

Since we have already used `OnPropertyChanged`, move `ActualThemeVariantProperty` changed handler into it

Signed-off-by: leo <[email protected]>

Author: love-linger

src/Views/ChangeStatusIcon.cs

@@ -42,11 +42,6 @@ public Models.Change Change
             set => SetValue(ChangeProperty, value);
         }
 
-        public ChangeStatusIcon()
-        {
-            ActualThemeVariantChanged += (_, _) => InvalidateVisual();
-        }
-
         public override void Render(DrawingContext context)
         {
             if (Change == null || Bounds.Width <= 0)
@@ -87,7 +82,9 @@ protected override void OnPropertyChanged(AvaloniaPropertyChangedEventArgs chang
         {
             base.OnPropertyChanged(change);
 
-            if (change.Property == IsUnstagedChangeProperty || change.Property == ChangeProperty)
+            if (change.Property == IsUnstagedChangeProperty || 
+                change.Property == ChangeProperty ||
+                (change.Property.Name == "ActualThemeVariant" && change.NewValue != null))
                 InvalidateVisual();
         }
     }