* fixed a bug, which caused the active player indicator to be always …

…on blue side
Johannes-Schneider · Oct 26, 2020 · 80abcd7 · 80abcd7
1 parent 3d95e62
commit 80abcd7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/GoldDiff/View/Controller/GoldDifferenceWindowController.cs b/GoldDiff/View/Controller/GoldDifferenceWindowController.cs
@@ -66,22 +66,22 @@ private void UpdateActivePlayerBackground()
                 Model.TopPlayerBackground = Model.ActivePlayerOnBlueSideBackground;
                 return;
             }
-            if (Model.JunglePlayerBlueSide?.IsActivePlayer == true || Model.JunglePlayerRedSide?.IsActivePlayer == true)
+            if (Model.JunglePlayerBlueSide?.IsActivePlayer == true)
             {
                 Model.JunglePlayerBackground = Model.ActivePlayerOnBlueSideBackground;
                 return;
             }
-            if (Model.MiddlePlayerBlueSide?.IsActivePlayer == true || Model.MiddlePlayerRedSide?.IsActivePlayer == true)
+            if (Model.MiddlePlayerBlueSide?.IsActivePlayer == true)
             {
                 Model.MiddlePlayerBackground = Model.ActivePlayerOnBlueSideBackground;
                 return;
             }
-            if (Model.BottomPlayerBlueSide?.IsActivePlayer == true || Model.BottomPlayerRedSide?.IsActivePlayer == true)
+            if (Model.BottomPlayerBlueSide?.IsActivePlayer == true)
             {
                 Model.BottomPlayerBackground = Model.ActivePlayerOnBlueSideBackground;
                 return;
             }
-            if (Model.SupportPlayerBlueSide?.IsActivePlayer == true || Model.SupportPlayerRedSide?.IsActivePlayer == true)
+            if (Model.SupportPlayerBlueSide?.IsActivePlayer == true)
             {
                 Model.SupportPlayerBackground = Model.ActivePlayerOnBlueSideBackground;
                 return;
@@ -91,22 +91,22 @@ private void UpdateActivePlayerBackground()
                 Model.TopPlayerBackground = Model.ActivePlayerOnRedSideBackground;
                 return;
             }
-            if (Model.JunglePlayerRedSide?.IsActivePlayer == true || Model.JunglePlayerRedSide?.IsActivePlayer == true)
+            if (Model.JunglePlayerRedSide?.IsActivePlayer == true)
             {
                 Model.JunglePlayerBackground = Model.ActivePlayerOnRedSideBackground;
                 return;
             }
-            if (Model.MiddlePlayerRedSide?.IsActivePlayer == true || Model.MiddlePlayerRedSide?.IsActivePlayer == true)
+            if (Model.MiddlePlayerRedSide?.IsActivePlayer == true)
             {
                 Model.MiddlePlayerBackground = Model.ActivePlayerOnRedSideBackground;
                 return;
             }
-            if (Model.BottomPlayerRedSide?.IsActivePlayer == true || Model.BottomPlayerRedSide?.IsActivePlayer == true)
+            if (Model.BottomPlayerRedSide?.IsActivePlayer == true)
             {
                 Model.BottomPlayerBackground = Model.ActivePlayerOnRedSideBackground;
                 return;
             }
-            if (Model.SupportPlayerRedSide?.IsActivePlayer == true || Model.SupportPlayerRedSide?.IsActivePlayer == true)
+            if (Model.SupportPlayerRedSide?.IsActivePlayer == true)
             {
                 Model.SupportPlayerBackground = Model.ActivePlayerOnRedSideBackground;
                 return;

diff --git a/GoldDiff/View/GoldDifferenceWindow.xaml b/GoldDiff/View/GoldDifferenceWindow.xaml
@@ -28,6 +28,7 @@
             <GradientStop Offset="0.45" Color="Transparent" />
         </LinearGradientBrush>
         <LinearGradientBrush x:Key="{x:Static local:GoldDifferenceWindow.ActivePlayerOnRedSideBackground}" StartPoint="0, 0.5" EndPoint="1, 0.5" Opacity="0.35">
+            <GradientStop Offset="0" Color="Transparent" />
             <GradientStop Offset="0.55" Color="Transparent" />
             <GradientStop Offset="1" Color="{DynamicResource {x:Static view:GoldDiffSharedResourceKeys.LoLActivePlayerRegularColor}}" />
         </LinearGradientBrush>