refactor: 重构代码精简代码

Author: ArgoZhang

src/BootstrapBlazor/Components/Button/Button.razor

@@ -2,9 +2,13 @@
 @inherits ButtonBase
 
 <button @attributes="@AdditionalAttributes" type="@ButtonType.ToDescriptionString()" @onclick="@OnClickButton" id="@Id" class="@ClassName" disabled="@Disabled" role="button" data-bs-placement="@PlacementString" data-bs-trigger="@TriggerString" aria-disabled="@DisabledString" tabindex="@Tab" @onclick:stopPropagation="@StopPropagation" @ref="ButtonElement">
-    @if (!string.IsNullOrEmpty(ButtonIcon))
+    @if(IsAsyncLoading)
     {
-        <i class="@ButtonIcon"></i>
+        <i class="@LoadingIcon"></i>
+    }
+    else if (!string.IsNullOrEmpty(Icon))
+    {
+        <i class="@Icon"></i>
     }
     @if (!string.IsNullOrEmpty(Text))
     {

src/BootstrapBlazor/Components/Button/Button.razor.cs

@@ -49,7 +49,6 @@ protected virtual async Task OnClickButton()
         if (IsAsync && ButtonType == ButtonType.Button)
         {
             IsAsyncLoading = true;
-            ButtonIcon = LoadingIcon;
             IsDisabled = true;
         }
 
@@ -65,7 +64,6 @@ protected virtual async Task OnClickButton()
         // 恢复按钮
         if (IsAsync && ButtonType == ButtonType.Button)
         {
-            ButtonIcon = Icon;
             IsDisabled = IsKeepDisabled;
             IsAsyncLoading = false;
         }

src/BootstrapBlazor/Components/Button/ButtonBase.cs

@@ -42,11 +42,6 @@ public abstract class ButtonBase : TooltipWrapperBase
     /// </summary>
     protected string? Tab => IsDisabled ? "-1" : null;
 
-    /// <summary>
-    /// 获得/设置 实际按钮渲染图标
-    /// </summary>
-    protected string? ButtonIcon { get; set; }
-
     /// <summary>
     /// 获得/设置 按钮风格枚举
     /// </summary>
@@ -173,8 +168,6 @@ protected override void OnInitialized()
     {
         base.OnInitialized();
 
-        ButtonIcon = Icon;
-
         if (IsAsync && ValidateForm != null)
         {
             // 开启异步操作时与 ValidateForm 联动
@@ -191,11 +184,6 @@ protected override void OnParametersSet()
 
         LoadingIcon ??= IconTheme.GetIconByKey(ComponentIcons.ButtonLoadingIcon);
 
-        if (!IsAsyncLoading)
-        {
-            ButtonIcon = Icon;
-        }
-
         if (Tooltip != null && !string.IsNullOrEmpty(TooltipText))
         {
             Tooltip.SetParameters(TooltipText, TooltipPlacement, TooltipTrigger);
@@ -260,7 +248,6 @@ public void SetDisable(bool disable)
     internal void TriggerAsync(bool loading)
     {
         IsAsyncLoading = loading;
-        ButtonIcon = loading ? LoadingIcon : Icon;
         SetDisable(loading);
     }
 

src/BootstrapBlazor/Components/Button/CountButton.cs

@@ -34,14 +34,12 @@ public class CountButton : Button
     protected override async Task OnClickButton()
     {
         IsAsyncLoading = true;
-        ButtonIcon = LoadingIcon;
         IsDisabled = true;
 
         await Task.Run(() => InvokeAsync(HandlerClick));
         await UpdateCount();
 
         IsDisabled = false;
-        ButtonIcon = Icon;
         IsAsyncLoading = false;
     }
 
@@ -65,7 +63,6 @@ private async Task UpdateCount()
     {
         var count = Count;
         var text = Text;
-        ButtonIcon = null;
         while (count > 0)
         {
             Text = GetCountText(count--, text);

src/BootstrapBlazor/Components/Button/ExportPdfButton.cs

@@ -83,7 +83,6 @@ protected override void OnParametersSet()
         base.OnParametersSet();
 
         Icon ??= IconTheme.GetIconByKey(ComponentIcons.TableExportPdfIcon);
-        ButtonIcon = Icon;
     }
 
     /// <summary>

src/BootstrapBlazor/Components/Button/PopConfirmButton.razor

@@ -17,9 +17,13 @@ else
     @<DynamicElement TagName="@TagName" OnClick="Show" id="@Id"
                      class="@ClassString" role="dialog" tabindex="@Tab" data-bb-confirm="@ConfirmString"
                      data-bs-custom-class="@CustomClassString" data-bs-trigger="@TriggerString" data-bs-placement="@PlacementString">
-        @if (!string.IsNullOrEmpty(ButtonIcon))
+        @if(IsAsyncLoading)
         {
-            <i class="@ButtonIcon"></i>
+            <i class="@LoadingIcon"></i>
+        }
+        else if (!string.IsNullOrEmpty(Icon))
+        {
+            <i class="@Icon"></i>
         }
         @if (!string.IsNullOrEmpty(Text))
         {

src/BootstrapBlazor/Components/Button/PopConfirmButton.razor.cs

@@ -95,7 +95,7 @@ private async Task OnClickConfirm()
         if (IsAsync)
         {
             IsDisabled = true;
-            ButtonIcon = LoadingIcon;
+            IsAsyncLoading = true;
             StateHasChanged();
             await Task.Run(() => InvokeAsync(OnConfirm));
 
@@ -106,7 +106,7 @@ private async Task OnClickConfirm()
             else
             {
                 IsDisabled = false;
-                ButtonIcon = Icon;
+                IsAsyncLoading = false;
                 StateHasChanged();
             }
         }

src/BootstrapBlazor/Components/Print/PrintButton.razor

@@ -3,9 +3,9 @@
 @attribute [BootstrapModuleAutoLoader("Print/PrintButton.razor.js")]
 
 <a @attributes="@AdditionalAttributes" id="@Id" href="@PreviewUrl" class="@ClassName" disabled="@Disabled" role="button" aria-disabled="@DisabledString" tabindex="@Tab">
-    @if (!string.IsNullOrEmpty(ButtonIcon))
+    @if (!string.IsNullOrEmpty(Icon))
     {
-        <i class="@ButtonIcon"></i>
+        <i class="@Icon"></i>
     }
     @if (!string.IsNullOrEmpty(Text))
     {

src/BootstrapBlazor/Components/Print/PrintButton.razor.cs

@@ -40,6 +40,5 @@ protected override void OnParametersSet()
         base.OnParametersSet();
 
         Icon ??= IconTheme.GetIconByKey(ComponentIcons.PrintButtonIcon);
-        ButtonIcon = Icon;
     }
 }