Skip to content

Commit 1642c81

Browse files
authored
Fix(Fluent): TextBox invalidation style (#10887)
* Fix TextBox invalidation style * Update Common theme files * Update resource name
1 parent d225c5e commit 1642c81

File tree

6 files changed

+40
-0
lines changed

6 files changed

+40
-0
lines changed

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Resources/Variables.xaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
<Thickness x:Key="DatePickerHostPadding">0,1,0,2</Thickness>
4545
<Thickness x:Key="DatePickerHostMonthPadding">9,0,0,1</Thickness>
4646
<Thickness x:Key="ComboBoxEditableTextPadding">10,0,30,0</Thickness>
47+
<Thickness x:Key="TextControlErrorBorderPadding">1</Thickness>
4748

4849
<system:Double x:Key="ComboBoxMinHeight">24</system:Double>
4950
<!-- <Thickness x:Key="ComboBoxPadding">12,1,0,3</Thickness> -->

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Styles/TextBox.xaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,8 +233,19 @@
233233
</ControlTemplate.Triggers>
234234
</ControlTemplate>
235235

236+
<ControlTemplate x:Key="DefaultTextBoxInvalidationStyle">
237+
<Border
238+
BorderBrush="{DynamicResource SystemFillColorCriticalBrush}"
239+
BorderThickness="{DynamicResource TextControlBorderThemeThickness}"
240+
CornerRadius="{DynamicResource ControlCornerRadius}"
241+
Padding="{StaticResource TextControlErrorBorderPadding}">
242+
<AdornedElementPlaceholder />
243+
</Border>
244+
</ControlTemplate>
245+
236246
<Style x:Key="DefaultTextBoxStyle" TargetType="{x:Type TextBox}">
237247
<Setter Property="FocusVisualStyle" Value="{x:Null}" />
248+
<Setter Property="Validation.ErrorTemplate" Value="{DynamicResource DefaultTextBoxInvalidationStyle}" />
238249
<Setter Property="ContextMenu" Value="{DynamicResource DefaultControlContextMenu}" />
239250
<Setter Property="Foreground" Value="{DynamicResource TextControlForeground}" />
240251
<Setter Property="CaretBrush" Value="{DynamicResource TextControlForeground}" />

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.Dark.xaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
<Thickness x:Key="DatePickerHostPadding">0,1,0,2</Thickness>
7979
<Thickness x:Key="DatePickerHostMonthPadding">9,0,0,1</Thickness>
8080
<Thickness x:Key="ComboBoxEditableTextPadding">10,0,30,0</Thickness>
81+
<Thickness x:Key="TextControlErrorBorderPadding">1</Thickness>
8182
<system:Double x:Key="ComboBoxMinHeight">24</system:Double>
8283
<!-- <Thickness x:Key="ComboBoxPadding">12,1,0,3</Thickness> -->
8384
<system:Double x:Key="NavigationViewItemOnLeftMinHeight">32</system:Double>
@@ -4959,8 +4960,14 @@
49594960
</Trigger>
49604961
</ControlTemplate.Triggers>
49614962
</ControlTemplate>
4963+
<ControlTemplate x:Key="DefaultTextBoxInvalidationStyle">
4964+
<Border BorderBrush="{DynamicResource SystemFillColorCriticalBrush}" BorderThickness="{DynamicResource TextControlBorderThemeThickness}" CornerRadius="{DynamicResource ControlCornerRadius}" Padding="{StaticResource TextControlErrorBorderPadding}">
4965+
<AdornedElementPlaceholder />
4966+
</Border>
4967+
</ControlTemplate>
49624968
<Style x:Key="DefaultTextBoxStyle" TargetType="{x:Type TextBox}">
49634969
<Setter Property="FocusVisualStyle" Value="{x:Null}" />
4970+
<Setter Property="Validation.ErrorTemplate" Value="{DynamicResource DefaultTextBoxInvalidationStyle}" />
49644971
<Setter Property="ContextMenu" Value="{DynamicResource DefaultControlContextMenu}" />
49654972
<Setter Property="Foreground" Value="{DynamicResource TextControlForeground}" />
49664973
<Setter Property="CaretBrush" Value="{DynamicResource TextControlForeground}" />

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.HC.xaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
<Thickness x:Key="DatePickerHostPadding">0,1,0,2</Thickness>
7979
<Thickness x:Key="DatePickerHostMonthPadding">9,0,0,1</Thickness>
8080
<Thickness x:Key="ComboBoxEditableTextPadding">10,0,30,0</Thickness>
81+
<Thickness x:Key="TextControlErrorBorderPadding">1</Thickness>
8182
<system:Double x:Key="ComboBoxMinHeight">24</system:Double>
8283
<!-- <Thickness x:Key="ComboBoxPadding">12,1,0,3</Thickness> -->
8384
<system:Double x:Key="NavigationViewItemOnLeftMinHeight">32</system:Double>
@@ -4872,8 +4873,14 @@
48724873
</Trigger>
48734874
</ControlTemplate.Triggers>
48744875
</ControlTemplate>
4876+
<ControlTemplate x:Key="DefaultTextBoxInvalidationStyle">
4877+
<Border BorderBrush="{DynamicResource SystemFillColorCriticalBrush}" BorderThickness="{DynamicResource TextControlBorderThemeThickness}" CornerRadius="{DynamicResource ControlCornerRadius}" Padding="{StaticResource TextControlErrorBorderPadding}">
4878+
<AdornedElementPlaceholder />
4879+
</Border>
4880+
</ControlTemplate>
48754881
<Style x:Key="DefaultTextBoxStyle" TargetType="{x:Type TextBox}">
48764882
<Setter Property="FocusVisualStyle" Value="{x:Null}" />
4883+
<Setter Property="Validation.ErrorTemplate" Value="{DynamicResource DefaultTextBoxInvalidationStyle}" />
48774884
<Setter Property="ContextMenu" Value="{DynamicResource DefaultControlContextMenu}" />
48784885
<Setter Property="Foreground" Value="{DynamicResource TextControlForeground}" />
48794886
<Setter Property="CaretBrush" Value="{DynamicResource TextControlForeground}" />

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.Light.xaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
<Thickness x:Key="DatePickerHostPadding">0,1,0,2</Thickness>
7979
<Thickness x:Key="DatePickerHostMonthPadding">9,0,0,1</Thickness>
8080
<Thickness x:Key="ComboBoxEditableTextPadding">10,0,30,0</Thickness>
81+
<Thickness x:Key="TextControlErrorBorderPadding">1</Thickness>
8182
<system:Double x:Key="ComboBoxMinHeight">24</system:Double>
8283
<!-- <Thickness x:Key="ComboBoxPadding">12,1,0,3</Thickness> -->
8384
<system:Double x:Key="NavigationViewItemOnLeftMinHeight">32</system:Double>
@@ -4968,8 +4969,14 @@
49684969
</Trigger>
49694970
</ControlTemplate.Triggers>
49704971
</ControlTemplate>
4972+
<ControlTemplate x:Key="DefaultTextBoxInvalidationStyle">
4973+
<Border BorderBrush="{DynamicResource SystemFillColorCriticalBrush}" BorderThickness="{DynamicResource TextControlBorderThemeThickness}" CornerRadius="{DynamicResource ControlCornerRadius}" Padding="{StaticResource TextControlErrorBorderPadding}">
4974+
<AdornedElementPlaceholder />
4975+
</Border>
4976+
</ControlTemplate>
49714977
<Style x:Key="DefaultTextBoxStyle" TargetType="{x:Type TextBox}">
49724978
<Setter Property="FocusVisualStyle" Value="{x:Null}" />
4979+
<Setter Property="Validation.ErrorTemplate" Value="{DynamicResource DefaultTextBoxInvalidationStyle}" />
49734980
<Setter Property="ContextMenu" Value="{DynamicResource DefaultControlContextMenu}" />
49744981
<Setter Property="Foreground" Value="{DynamicResource TextControlForeground}" />
49754982
<Setter Property="CaretBrush" Value="{DynamicResource TextControlForeground}" />

src/Microsoft.DotNet.Wpf/src/Themes/PresentationFramework.Fluent/Themes/Fluent.xaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@
7878
<Thickness x:Key="DatePickerHostPadding">0,1,0,2</Thickness>
7979
<Thickness x:Key="DatePickerHostMonthPadding">9,0,0,1</Thickness>
8080
<Thickness x:Key="ComboBoxEditableTextPadding">10,0,30,0</Thickness>
81+
<Thickness x:Key="TextControlErrorBorderPadding">1</Thickness>
8182
<system:Double x:Key="ComboBoxMinHeight">24</system:Double>
8283
<!-- <Thickness x:Key="ComboBoxPadding">12,1,0,3</Thickness> -->
8384
<system:Double x:Key="NavigationViewItemOnLeftMinHeight">32</system:Double>
@@ -4218,8 +4219,14 @@
42184219
</Trigger>
42194220
</ControlTemplate.Triggers>
42204221
</ControlTemplate>
4222+
<ControlTemplate x:Key="DefaultTextBoxInvalidationStyle">
4223+
<Border BorderBrush="{DynamicResource SystemFillColorCriticalBrush}" BorderThickness="{DynamicResource TextControlBorderThemeThickness}" CornerRadius="{DynamicResource ControlCornerRadius}" Padding="{StaticResource TextControlErrorBorderPadding}">
4224+
<AdornedElementPlaceholder />
4225+
</Border>
4226+
</ControlTemplate>
42214227
<Style x:Key="DefaultTextBoxStyle" TargetType="{x:Type TextBox}">
42224228
<Setter Property="FocusVisualStyle" Value="{x:Null}" />
4229+
<Setter Property="Validation.ErrorTemplate" Value="{DynamicResource DefaultTextBoxInvalidationStyle}" />
42234230
<Setter Property="ContextMenu" Value="{DynamicResource DefaultControlContextMenu}" />
42244231
<Setter Property="Foreground" Value="{DynamicResource TextControlForeground}" />
42254232
<Setter Property="CaretBrush" Value="{DynamicResource TextControlForeground}" />

0 commit comments

Comments
 (0)