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

Show file tree
Hide file tree
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
Minor cleanup
  • Loading branch information
@nicolaihenriksen
nicolaihenriksen committed Feb 17, 2023
commit 0811b7caa925f3603b7fc543f2a4a37c960eea5c
4 changes: 2 additions & 2 deletions MaterialDesignThemes.UITests/WPF/ComboBoxes/ComboBoxTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ public async Task ComboBox_WithHintAndHelperText_RespectsPadding(string styleNam
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down Expand Up @@ -224,7 +224,7 @@ public async Task ComboBox_WithHintAndValidationError_RespectsPadding(string sty
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down
4 changes: 2 additions & 2 deletions MaterialDesignThemes.UITests/WPF/DatePickers/DatePickerTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ public async Task DatePicker_WithHintAndHelperText_RespectsPadding(string styleN
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down Expand Up @@ -170,7 +170,7 @@ public async Task DatePicker_WithHintAndValidationError_RespectsPadding(string s
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down
4 changes: 2 additions & 2 deletions MaterialDesignThemes.UITests/WPF/PasswordBoxes/PasswordBoxTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ public async Task PasswordBox_WithHintAndHelperText_RespectsPadding(string style
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down Expand Up @@ -262,7 +262,7 @@ public async Task PasswordBox_WithHintAndValidationError_RespectsPadding(string
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down
4 changes: 2 additions & 2 deletions MaterialDesignThemes.UITests/WPF/TextBoxes/TextBoxTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -488,7 +488,7 @@ public async Task TextBox_WithHintAndHelperText_RespectsPadding(string styleName
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down Expand Up @@ -529,7 +529,7 @@ public async Task TextBox_WithHintAndValidationError_RespectsPadding(string styl
await using var recorder = new TestRecorder(App);

// FIXME: Tolerance needed because TextFieldAssist.TextBoxViewMargin is in play and slightly modifies the hint text placement in certain cases.
double tolerance = 1.5;
const double tolerance = 1.5;

string styleAttribute = $"Style=\"{{StaticResource {styleName}}}\"";
string paddingAttribute = padding.HasValue ? $"Padding=\"{padding.Value}\"" : string.Empty;
Expand Down
2 changes: 1 addition & 1 deletion MaterialDesignThemes.Wpf/Converters/ThicknessCloneConverter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ public class ThicknessCloneConverter : IValueConverter
{
public ThicknessEdges CloneEdges { get; set; } = ThicknessEdges.All;

public double NonClonedEdgeValue { get; set; } = 0;
public double NonClonedEdgeValue { get; set; }

public double? FixedLeft { get; set; }
public double? FixedTop { get; set; }
Expand Down