feat(Search): redesign OnClear parameter (#5929)

* refactor: 移除 RenderTemplate 提高性能

* refactor: 重构清空逻辑

* doc: 更新示例

* test: 更新单元测试

* chore: bump version 9.5.13

* chore: bump version 6.0

Co-Authored-By: zpmzuus <[email protected]>

---------

Co-authored-by: zpmzuus <[email protected]>

Author: ArgoZhang

src/BootstrapBlazor.Server/Components/Pages/Coms.razor.cs

@@ -20,7 +20,7 @@ public sealed partial class Coms
         return Task.FromResult<IEnumerable<string?>>(ComponentItems.Where(i => i.Contains(searchText, StringComparison.OrdinalIgnoreCase)).ToList());
     }
 
-    private Task OnClear(string searchText)
+    private Task OnClear()
     {
         SearchText = "";
         StateHasChanged();

src/BootstrapBlazor.Server/Components/Samples/Searches.razor

@@ -23,8 +23,7 @@
            Name="DisplayButton">
     <Search PlaceHolder="@Localizer["SearchesPlaceHolder"]"
             ShowClearButton="true"
-            OnSearch="@OnDisplaySearch"
-            OnClear="@OnClear"></Search>
+            OnSearch="@OnDisplaySearch"></Search>
     <ConsoleLogger @ref="DisplayLogger"></ConsoleLogger>
 </DemoBlock>
 

src/BootstrapBlazor.Server/Components/Samples/Searches.razor.cs

@@ -40,12 +40,6 @@ private Task<IEnumerable<string>> OnDisplaySearch(string searchText)
         return Task.FromResult<IEnumerable<string>>([$"{searchText}1", $"{searchText}12", $"{searchText}123"]);
     }
 
-    private Task<IEnumerable<string>> OnClear(string searchText)
-    {
-        DisplayLogger.Log($"OnClear: {searchText}");
-        return Task.FromResult<IEnumerable<string>>([$"{searchText}1", $"{searchText}12", $"{searchText}123"]);
-    }
-
     [NotNull]
     private ConsoleLogger? KeyboardLogger { get; set; }
 

src/BootstrapBlazor/BootstrapBlazor.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Razor">
 
   <PropertyGroup>
-    <Version>9.5.13-beta01</Version>
+    <Version>9.6.0</Version>
   </PropertyGroup>
 
   <ItemGroup>

src/BootstrapBlazor/Components/EditorForm/EditorForm.razor

@@ -3,34 +3,32 @@
 @inherits BootstrapComponentBase
 
 <div class="@ClassString">
-    <CascadingValue Value="@_editorItems" IsFixed="true">
+    <CascadingValue Value="@_editorItems" IsFixed="false">
         @FieldItems?.Invoke(Model)
     </CascadingValue>
     <CascadingValue Value="this" Name="EditorForm">
-        <RenderTemplate>
-            @if (ShowUnsetGroupItemsOnTop)
+        @if (ShowUnsetGroupItemsOnTop)
+        {
+            if (UnsetGroupItems.Any())
             {
-                if (UnsetGroupItems.Any())
-                {
-                    @RenderUnsetGroupItems
-                }
-                @foreach (var g in GroupItems)
-                {
-                    @RenderGroupItems(g)
-                }
+                @RenderUnsetGroupItems
             }
-            else
+            @foreach (var g in GroupItems)
             {
-                @foreach (var g in GroupItems)
-                {
-                    @RenderGroupItems(g)
-                }
-                if (UnsetGroupItems.Any())
-                {
-                    @RenderUnsetGroupItems
-                }
+                @RenderGroupItems(g)
+            }
+        }
+        else
+        {
+            @foreach (var g in GroupItems)
+            {
+                @RenderGroupItems(g)
+            }
+            if (UnsetGroupItems.Any())
+            {
+                @RenderUnsetGroupItems
             }
-        </RenderTemplate>
+        }
     </CascadingValue>
 
     @if (Buttons != null)

src/BootstrapBlazor/Components/Search/Search.razor.cs

@@ -146,7 +146,7 @@ public partial class Search<TValue>
     /// Gets or sets the event callback when the clear button is clicked. Default is null.
     /// </summary>
     [Parameter]
-    public Func<string?, Task>? OnClear { get; set; }
+    public Func<Task>? OnClear { get; set; }
 
     [Inject]
     [NotNull]
@@ -221,18 +221,17 @@ private async Task OnSearchClick()
             {
                 await InvokeVoidAsync("showList", Id);
             }
-            StateHasChanged();
         }
     }
 
     private async Task OnClearClick()
     {
+        _displayText = "";
         if (OnClear != null)
         {
-            await OnClear(_displayText);
+            await OnClear();
         }
-        _displayText = "";
-        _filterItems = [];
+        await OnSearchClick();
     }
 
     private string? GetDisplayText(TValue item)
@@ -272,6 +271,7 @@ public async Task TriggerFilter(string val)
         if (IsTriggerSearchByInput)
         {
             await OnSearchClick();
+            StateHasChanged();
         }
         _dropdown.Render();
     }

test/UnitTest/Components/SearchTest.cs

@@ -138,7 +138,7 @@ public async Task OnClearClick_Ok()
             builder.Add(s => s.ClearButtonColor, Color.Secondary);
             builder.Add(s => s.ClearButtonIcon, "test-icon");
             builder.Add(s => s.ClearButtonText, "Clear");
-            builder.Add(s => s.OnClear, v =>
+            builder.Add(s => s.OnClear, () =>
             {
                 ret = true;
                 return Task.CompletedTask;