Colorpicker fixes (#9277)

Co-authored-by: Niels Laute <niels9001@hotmail.com>
microsoft · Jan 25, 2021 · 2e961d1 · 2e961d1
1 parent 5713230
commit 2e961d1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/src/modules/colorPicker/ColorPickerUI/Controls/ColorPickerControl.xaml b/src/modules/colorPicker/ColorPickerUI/Controls/ColorPickerControl.xaml
@@ -274,14 +274,14 @@
                     <Button x:Name="CancelButton"
                             Click="CancelButton_Click"
                             AutomationProperties.Name="{x:Static p:Resources.Cancel}"
-                            Content="Cancel"
+                            Content="{x:Static p:Resources.Cancel}"
                             Width="80"/>
                     <Button Margin="12,0,12,0"
                             x:Name="OKButton"
                             Click="OKButton_Click"
                             Style="{StaticResource AccentButtonStyle}"
                             AutomationProperties.Name="{x:Static p:Resources.OK}"
-                            Content="OK"
+                            Content="{x:Static p:Resources.OK}"
                             Width="80"/>
                 </WrapPanel>
             </StackPanel>

diff --git a/src/modules/colorPicker/ColorPickerUI/Controls/ColorPickerControl.xaml.cs b/src/modules/colorPicker/ColorPickerUI/Controls/ColorPickerControl.xaml.cs
@@ -199,6 +199,7 @@ private void ShowDetails()
                 CurrentColorButton.BeginAnimation(Button.MarginProperty, moveColor);
                 detailsStackPanel.BeginAnimation(StackPanel.OpacityProperty, opacityAppear);
                 detailsGrid.BeginAnimation(Grid.HeightProperty, resize);
+                CurrentColorButton.IsEnabled = false;
             }
         }
 
@@ -226,6 +227,7 @@ private void HideDetails()
                 detailsStackPanel.BeginAnimation(Window.OpacityProperty, opacityAppear);
                 detailsGrid.BeginAnimation(Grid.HeightProperty, resize);
                 detailsGrid.Visibility = Visibility.Collapsed;
+                CurrentColorButton.IsEnabled = true;
             }
         }
 

diff --git a/src/modules/colorPicker/ColorPickerUI/Resources/Styles.xaml b/src/modules/colorPicker/ColorPickerUI/Resources/Styles.xaml
@@ -120,15 +120,9 @@
                             <Setter TargetName="Border" Property="BorderBrush" Value="{DynamicResource ButtonBorderBrushPressed}" />
                             <Setter TargetName="ContentPresenter" Property="TextElement.Foreground" Value="{DynamicResource ButtonForegroundPressed}" />
                         </Trigger>
-                        <Trigger Property="IsEnabled" Value="False">
-                            <Setter TargetName="Background" Property="Background" Value="{DynamicResource ButtonBackgroundDisabled}" />
-                            <Setter TargetName="Border" Property="BorderBrush" Value="{DynamicResource ButtonBorderBrushDisabled}" />
-                            <Setter TargetName="ContentPresenter" Property="TextElement.Foreground" Value="{DynamicResource ButtonForegroundDisabled}" />
-                        </Trigger>
                     </ControlTemplate.Triggers>
                 </ControlTemplate>
             </Setter.Value>
         </Setter>
     </Style>
-
 </ResourceDictionary>