Skip to content

Commit b33cae0

Browse files
authored
refactor to remove MudWheel redundant declarations (#66)
1 parent 90802cf commit b33cae0

File tree

2 files changed

+20
-27
lines changed

2 files changed

+20
-27
lines changed

CodeBeam.MudExtensions/Components/DateWheelPicker/MudDateWheelPicker.razor

Lines changed: 19 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@
77
<MudInputControl Label="@Label" Variant="@Variant" HelperText="@HelperText" HelperTextOnFocus="@HelperTextOnFocus" FullWidth="@FullWidth" Margin="@Margin" Class="@Classname" Style="@Style"
88
Error="@Error" ErrorText="@ErrorText" ErrorId="@ErrorId" Disabled="@Disabled" @onclick="@(Editable ? null : ToggleMenu)" Required="@Required" ForId="@FieldId">
99
<InputContent>
10-
<MudInput
11-
@ref="InputReference" Margin="@Margin" Placeholder="@Placeholder"
10+
<MudInput @ref="InputReference" Margin="@Margin" Placeholder="@Placeholder"
1211
Variant="@Variant" InputType="InputType.Text"
1312
TextUpdateSuppression="false"
1413
Value="@(Text)" DisableUnderLine="@DisableUnderLine"
@@ -33,30 +32,28 @@
3332
}
3433
</div>
3534
}
36-
35+
3736
<div class="d-flex">
3837
@if (DateView == DateView.Date || DateView == DateView.Both)
3938
{
40-
int yearIndex = DateFormat.IndexOf('y');
41-
int monthIndex = DateFormat.IndexOf('M');
42-
int dayIndex = DateFormat.IndexOf('d');
43-
@if (yearIndex < dayIndex)
44-
{
45-
<MudWheel Class="mud-width-full" @bind-Value="_year" ItemCollection="Years" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Year : null)" Dense="@Dense" Color="@Color" Disabled="FixYear" />
46-
<MudWheel Class="mud-width-full" Value="_month" ValueChanged=@((int m) => OnMonthChanged(m)) ItemCollection="Months" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Month : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixMonth" />
47-
<MudWheel Class="mud-width-full" @bind-Value="_day" ItemCollection="Days" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Day : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixDay" />
48-
}
49-
else if (monthIndex < dayIndex && dayIndex < yearIndex )
50-
{
51-
<MudWheel Class="mud-width-full" Value="_month" ValueChanged=@((int m) => OnMonthChanged(m)) ItemCollection="Months" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Month : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixMonth" />
52-
<MudWheel Class="mud-width-full" @bind-Value="_day" ItemCollection="Days" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Day : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixDay" />
53-
<MudWheel Class="mud-width-full" @bind-Value="_year" ItemCollection="Years" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Year : null)" Dense="@Dense" Color="@Color" Disabled="FixYear" />
54-
}
55-
else
39+
string dateFormat = new string(DateFormat);
40+
41+
while (dateFormat.Length > 0)
5642
{
57-
<MudWheel Class="mud-width-full" @bind-Value="_day" ItemCollection="Days" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Day : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixDay" />
58-
<MudWheel Class="mud-width-full" Value="_month" ValueChanged=@((int m) => OnMonthChanged(m)) ItemCollection="Months" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Month : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixMonth" />
59-
<MudWheel Class="mud-width-full" @bind-Value="_year" ItemCollection="Years" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Year : null)" Dense="@Dense" Color="@Color" Disabled="FixYear" />
43+
switch (dateFormat[0])
44+
{
45+
case 'd':
46+
<MudWheel Class="mud-width-full" @bind-Value="_day" ItemCollection="Days" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Day : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixDay" />
47+
break;
48+
case 'M':
49+
<MudWheel Class="mud-width-full" Value="_month" ValueChanged=@((int m) => OnMonthChanged(m)) ItemCollection="Months" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Month : null)" ToStringFunc="new Func<int, string>(NumberToString)" Dense="@Dense" Color="@Color" Disabled="FixMonth" />
50+
break;
51+
case 'y':
52+
<MudWheel Class="mud-width-full" @bind-Value="_year" ItemCollection="Years" Sensitivity="@Sensitivity" Label="@(ShowHeader ? LocalizedStrings.Year : null)" Dense="@Dense" Color="@Color" Disabled="FixYear" />
53+
break;
54+
}
55+
56+
dateFormat = dateFormat.Replace(dateFormat[0].ToString(), string.Empty);
6057
}
6158
}
6259
@if (DateView == DateView.Time || DateView == DateView.Both)

CodeBeam.MudExtensions/Components/DateWheelPicker/MudDateWheelPicker.razor.cs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -376,11 +376,7 @@ protected bool HasSeconds()
376376

377377
protected string NumberToString(int val)
378378
{
379-
if (val < 10)
380-
{
381-
return $"0{val}";
382-
}
383-
return val.ToString();
379+
return $"{val:00}";
384380
}
385381

386382
/// <summary>

0 commit comments

Comments
 (0)