From dcdc52592c1a2b19fff45098cb6e843d126eaf7e Mon Sep 17 00:00:00 2001 From: leo Date: Mon, 16 Jun 2025 12:31:37 +0800 Subject: [PATCH] code_review: PR #1423 Since we have already used `OnPropertyChanged`, move `ActualThemeVariantProperty` changed handler into it Signed-off-by: leo --- src/Views/ChangeStatusIcon.cs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/Views/ChangeStatusIcon.cs b/src/Views/ChangeStatusIcon.cs index ea88dbb5..8d43acac 100644 --- a/src/Views/ChangeStatusIcon.cs +++ b/src/Views/ChangeStatusIcon.cs @@ -42,11 +42,6 @@ namespace SourceGit.Views set => SetValue(ChangeProperty, value); } - public ChangeStatusIcon() - { - ActualThemeVariantChanged += (_, _) => InvalidateVisual(); - } - public override void Render(DrawingContext context) { if (Change == null || Bounds.Width <= 0) @@ -87,7 +82,9 @@ namespace SourceGit.Views { 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(); } }