diff options
author | Bret Johnson <bret.johnson@microsoft.com> | 2020-01-22 23:29:10 +0300 |
---|---|---|
committer | Bret Johnson <bret.johnson@microsoft.com> | 2020-01-22 23:29:10 +0300 |
commit | bcac5c63f19a098b5bfdf335a5acaeba5350f6c9 (patch) | |
tree | 2ca85811c6a2d63e8a66b12e4fa53fc8520065fd /Xamarin.PropertyEditing.Windows | |
parent | 51d031eb955fc6155e91d55e0acab7bb31ef6faa (diff) |
[VS] Show focus via focus rect, not color, for choice buttons
This fixes AB#1003978
Previously the choice buttons (used for picking brush type) showed focus via a different background color. But that color difference wasn't sufficiently obvious, per a11y testing. Our UI designer suggested that using a focus rect instead would be better, which is also what XET does here. And it indeed does look pretty good, IMO.
Diffstat (limited to 'Xamarin.PropertyEditing.Windows')
-rw-r--r-- | Xamarin.PropertyEditing.Windows/Themes/Resources.xaml | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/Xamarin.PropertyEditing.Windows/Themes/Resources.xaml b/Xamarin.PropertyEditing.Windows/Themes/Resources.xaml index 263066e..3bf8a45 100644 --- a/Xamarin.PropertyEditing.Windows/Themes/Resources.xaml +++ b/Xamarin.PropertyEditing.Windows/Themes/Resources.xaml @@ -202,7 +202,7 @@ <Setter Property="BorderThickness" Value="1" /> <Setter Property="Padding" Value="4,2,4,2" /> <Setter Property="Margin" Value="1,0,0,0" /> - <Setter Property="FocusVisualStyle" Value="{x:Null}" /> + <Setter Property="FocusVisualStyle" Value="{DynamicResource GenericVisualFocusStyle}" /> <Setter Property="ToolTip" Value="{Binding Tooltip}" /> <Setter Property="AutomationProperties.Name" Value="{Binding Name}" /> <Setter Property="AutomationProperties.HelpText" Value="{Binding Tooltip}" /> @@ -225,9 +225,6 @@ <Setter Property="Background" Value="{DynamicResource ToggleItemMouseOverBackgroundBrush}" /> <Setter Property="Foreground" Value="{DynamicResource ToggleItemMouseOverForegroundBrush}" /> </Trigger> - <Trigger Property="IsFocused" Value="True"> - <Setter Property="Background" Value="{DynamicResource ToggleItemMouseOverBackgroundBrush}" /> - </Trigger> <Trigger Property="IsPressed" Value="True"> <Setter Property="BorderBrush" Value="{DynamicResource ToggleItemPressedBorderBrush}" /> <Setter Property="Background" Value="{DynamicResource ToggleItemPressedBackgroundBrush}" /> |