Skip to content

Cleanup: Aligning hint, helper text and validation error based on padding #3086

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Keboo merged 17 commits into from
Feb 19, 2023
Merged

Cleanup: Aligning hint, helper text and validation error based on padding #3086

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
7b2d33d
Fix padding issues with DatePicker and TimePicker
nicolaihenriksen Feb 13, 2023
7cc2b97
Extend SmartHint demo page with custom padding and helper text options
nicolaihenriksen Feb 13, 2023
30ced21
Enable toggling validation errors on/off on SmartHint demo page
nicolaihenriksen Feb 13, 2023
fc3cf2a
Add failing UI tests for TextBox
nicolaihenriksen Feb 14, 2023
7bf2082
Add failing UI tests for TextBox with validation error
nicolaihenriksen Feb 14, 2023
cb3350c
Fix positioning of helper text and validation error in TextBox
nicolaihenriksen Feb 14, 2023
f6626b7
Added UI test for PasswordBox (some failing some passing)
nicolaihenriksen Feb 14, 2023
69d6ba2
Fix helper text alignment for PasswordBox with custom padding
nicolaihenriksen Feb 14, 2023
3662cb0
Add UI test for ComboBox
nicolaihenriksen Feb 17, 2023
1057751
Remove wrong usage of TextFieldAssist.TextBoxViewMargin
nicolaihenriksen Feb 17, 2023
02cc4a6
Add DatePicker and TimePicker to SmartHint demo page
nicolaihenriksen Feb 17, 2023
1e1647d
Add failing DatePicker UI tests
nicolaihenriksen Feb 17, 2023
f099401
Fix helper text placement in DatePicker
nicolaihenriksen Feb 17, 2023
876aa85
Add validation error to DatePicker/TimePicker in SmartHint demo page
nicolaihenriksen Feb 17, 2023
68a8ee7
Add failing TimePicker UI tests
nicolaihenriksen Feb 17, 2023
37d2222
Fix helper text placement in TimePicker
nicolaihenriksen Feb 17, 2023
0811b7c
Minor cleanup
nicolaihenriksen Feb 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix helper text placement in TimePicker
  • Loading branch information
@nicolaihenriksen
nicolaihenriksen committed Feb 17, 2023
commit 37d2222966a7cc986223c57fb0f0bf8975a1304b
5 changes: 2 additions & 3 deletions MaterialDesignThemes.Wpf/Themes/MaterialDesignTheme.TimePicker.xaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
<converters:FloatingHintOffsetCalculationConverter x:Key="FloatingHintOffsetCalculationConverter" />
<converters:PickerInnerPaddingConverter x:Key="PickerInnerPaddingConverter" />
<converters:OutlinedDateTimePickerActiveBorderThicknessConverter x:Key="OutlinedDateTimePickerActiveBorderThicknessConverter" />
<converters:ThicknessCloneConverter x:Key="HelperTextMarginConverter" CloneEdges="Left" />

<Style x:Key="MaterialDesignTimePickerTextBox" TargetType="{x:Type wpf:TimePickerTextBox}">
<Setter Property="AllowDrop" Value="true" />
Expand Down Expand Up @@ -207,7 +208,7 @@
Background="{TemplateBinding wpf:TextFieldAssist.UnderlineBrush}"
CornerRadius="{TemplateBinding wpf:TextFieldAssist.UnderlineCornerRadius}"
Visibility="{TemplateBinding wpf:TextFieldAssist.DecorationVisibility}" />
<Canvas x:Name="HelperTextWrapper" VerticalAlignment="Bottom">
<Canvas x:Name="HelperTextWrapper" VerticalAlignment="Bottom" Margin="{TemplateBinding Padding, Converter={StaticResource HelperTextMarginConverter}}">
<TextBlock x:Name="HelperTextTextBlock" Canvas.Top="2"
MaxWidth="{Binding ActualWidth, ElementName=border}"
FontSize="{TemplateBinding wpf:HintAssist.HelperTextFontSize}"
Expand Down Expand Up @@ -252,13 +253,11 @@
</MultiTrigger>
<Trigger Property="wpf:TextFieldAssist.HasFilledTextField" Value="True">
<Setter Property="Background" Value="{DynamicResource MaterialDesignTextFieldBoxBackground}" />
<Setter TargetName="HelperTextWrapper" Property="Margin" Value="16,0,0,0" />
</Trigger>
<Trigger Property="wpf:TextFieldAssist.HasOutlinedTextField" Value="True">
<Setter Property="BorderBrush" Value="{DynamicResource MaterialDesignTextAreaBorder}" />
<Setter Property="BorderThickness" Value="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=(wpf:TimePickerAssist.OutlinedBorderInactiveThickness)}" />
<Setter Property="VerticalContentAlignment" Value="Top" />
<Setter TargetName="HelperTextWrapper" Property="Margin" Value="16,0,0,0" />
<Setter TargetName="Hint" Property="FloatingOffset">
<Setter.Value>
<MultiBinding Converter="{StaticResource FloatingHintOffsetCalculationConverter}">
Expand Down