ChipField Fix Regression (#530)

Author: mckaragoz

CodeBeam.MudBlazor.Extensions.Docs/Pages/VisualTest.razor

@@ -1,17 +1,19 @@
 @page "/visualtest"
 
-<MudStack>
+<MudContainer Class="gap-4" MaxWidth="MaxWidth.Medium">
     <MudStack Row="true">
-        <MudSelect T="string" />
-        <MudSelectExtended T="string" />
-    </MudStack>
-    <MudStack Row="true">
-        <MudSelect T="string" Variant="Variant.Outlined" />
-        <MudSelectExtended T="string" Variant="Variant.Outlined" />
-    </MudStack>
-    <MudStack Row="true">
-        <MudSelect T="string" Variant="Variant.Filled" />
-        <MudSelectExtended T="string" Variant="Variant.Filled" />
+        <MudSelect T="Color" Margin="@(_margin ? Margin.Dense : Margin.Normal)" Dense="@_dense" Variant="@_variant" Label="Label" Clearable>
+            @foreach (var item in Enum.GetValues<Color>())
+            {
+                <MudSelectItem Value="@item">@item</MudSelectItem>
+            }
+        </MudSelect>
+        <MudSelectExtended T="Color" Margin="@(_margin ? Margin.Dense : Margin.Normal)" Dense="@_dense" Variant="@_variant" Label="Label" Clearable>
+            @foreach (var item in Enum.GetValues<Color>())
+            {
+                <MudSelectItemExtended Value="@item">@item</MudSelectItemExtended>
+            }
+        </MudSelectExtended>
     </MudStack>
 
     <MudStack Row="true">
@@ -26,5 +28,20 @@
         <MudTextField T="string" Variant="Variant.Filled" />
         <MudTextFieldExtended T="string" Variant="Variant.Filled" />
     </MudStack>
-</MudStack>
+    <MudStack>
+        <MudSwitchM3 @bind-Value="_dense" Label="Dense" />
+        <MudSwitchM3 @bind-Value="_margin" Label="Margin" />
+        <MudSelectExtended @bind-Value="_variant">
+            @foreach (var item in Enum.GetValues<Variant>())
+            {
+                <MudSelectItemExtended Value="@item">@item</MudSelectItemExtended>
+            }
+        </MudSelectExtended>
+    </MudStack>
+</MudContainer>
 
+@code {
+    private bool _dense;
+    private bool _margin;
+    private Variant _variant;
+}

CodeBeam.MudBlazor.Extensions.UnitTests/Components/ChipFieldTests.cs

@@ -0,0 +1,26 @@
+using MudExtensions.Docs.Examples;
+using FluentAssertions;
+using Bunit;
+using Microsoft.AspNetCore.Components;
+using Microsoft.AspNetCore.Components.Web;
+
+namespace MudExtensions.UnitTests.Components
+{
+    [TestFixture]
+    public class ChipFieldTests : BunitTest
+    {
+        [Test]
+        public async Task ChipFieldBasicTest()
+        {
+            var comp = Context.RenderComponent<MudChipField<string>>(opt =>
+            {
+                opt.Add(a => a.Values, new List<string> { "asdf", "asd" });
+            });
+            var field = comp.FindComponent<MudTextFieldExtended<string>>();
+            field.Find("input").Input(new ChangeEventArgs() { Value = "sdfg" });
+            await comp.InvokeAsync(() => comp.Instance.HandleKeyDown(new KeyboardEventArgs() { Key = " " }));
+            comp.Instance.Values.Should().BeEquivalentTo(new List<string> { "asdf", "asd", "sdfg" });
+            comp.Instance.Value.Should().BeEquivalentTo(null);
+        }
+    }
+}

CodeBeam.MudBlazor.Extensions/Components/ChipField/MudChipField.razor

@@ -42,8 +42,7 @@
                       Variant="@Variant"
                       @bind-Value="@_internalValue"
                       TextChanged="@(async() => await TextChanged.InvokeAsync())"
-                      ShowVisualiser="@(Values?.Any() ?? false)"
-                      Mask="@Mask">
+                      ShowVisualiser="@(Values?.Any() ?? false)">
 
     <DataVisualiser>
         <MudChipSet T="T" Class="@ChipClassname" Style="@ChipStylename" AllClosable="@Closeable" OnClose="Closed">

CodeBeam.MudBlazor.Extensions/Components/ChipField/MudChipField.razor.cs

@@ -120,7 +120,7 @@ public partial class MudChipField<T> : MudTextFieldExtended<T>
         /// </summary>
         /// <param name="args"></param>
         /// <returns></returns>
-        protected async Task HandleKeyDown(KeyboardEventArgs args)
+        protected internal async Task HandleKeyDown(KeyboardEventArgs args)
         {
             var result = args.Key;
             if (result.Equals(Delimiter, StringComparison.InvariantCultureIgnoreCase) && _internalValue != null)