perf(PopConfirmButton): remove dismiss event handler to improve performance (#5453)

* refactor: 精简代码

* refactor: 移除 dismiss 处理方法

* chore: bump version 9.3.1-beta39

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

---------

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

Author: ArgoZhang

src/BootstrapBlazor/BootstrapBlazor.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk.Razor">
 
   <PropertyGroup>
-    <Version>9.3.1-beta38</Version>
+    <Version>9.3.1-beta39</Version>
   </PropertyGroup>
 
   <ItemGroup>

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

@@ -5,7 +5,6 @@ import EventHandler from "../../modules/event-handler.js"
 
 const config = {
     class: 'popover-confirm',
-    dismiss: '.popover-confirm-buttons > div',
     popoverSelector: '.popover-confirm.show'
 }
 
@@ -63,22 +62,6 @@ export function init(id) {
     EventHandler.on(el, 'inserted.bs.popover', confirm.inserted)
     EventHandler.on(el, 'hide.bs.popover', confirm.hide)
 
-    if (config.dismiss != null) {
-        confirm.dismissHandler = e => {
-            const ele = e.target.closest(config.popoverSelector)
-            if (ele) {
-                const element = getDescribedOwner(ele)
-                if (element) {
-                    const popover = bootstrap.Popover.getInstance(element);
-                    if (popover) {
-                        popover.hide()
-                    }
-                }
-            }
-        }
-        EventHandler.on(document, 'click', config.dismiss, confirm.dismissHandler)
-    }
-
     confirm.checkCancel = el => {
         // check button
         let self = el === confirm.el || el.closest('.dropdown-toggle') === confirm.el
@@ -170,9 +153,6 @@ export function dispose(id) {
         if (confirm.popover) {
             confirm.popover.dispose();
         }
-        if (config.dismiss) {
-            EventHandler.off(document, 'click', config.dismiss, confirm.dismissHandler)
-        }
     }
 }
 

src/BootstrapBlazor/Components/Button/PopConfirmButtonContent.razor

@@ -10,8 +10,8 @@
         <div class="popover-confirm-body">
             @if (ChildContent != null)
             {
-                <CascadingValue Value="TriggerCloseAsync" Name="PopoverConfirmButtonCloseAsync" IsFixed="true">
-                    <CascadingValue Value="TriggerConfirmAsync" Name="PopoverConfirmButtonConfirmAsync" IsFixed="true">
+                <CascadingValue Value="OnCloseClick" Name="PopoverConfirmButtonCloseAsync" IsFixed="true">
+                    <CascadingValue Value="OnConfirmClick" Name="PopoverConfirmButtonConfirmAsync" IsFixed="true">
                         @ChildContent
                     </CascadingValue>
                 </CascadingValue>

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

@@ -121,6 +121,7 @@ protected override void OnParametersSet()
     /// </summary>
     public async Task OnCloseClick()
     {
+        await InvokeVoidAsync("hide", Id);
         if (OnClose != null)
         {
             await OnClose();
@@ -132,21 +133,10 @@ public async Task OnCloseClick()
     /// </summary>
     public async Task OnConfirmClick()
     {
+        await InvokeVoidAsync("hide", Id);
         if (OnConfirm != null)
         {
             await OnConfirm();
         }
     }
-
-    private async Task TriggerCloseAsync()
-    {
-        await InvokeVoidAsync("hide", Id);
-        await OnCloseClick();
-    }
-
-    private async Task TriggerConfirmAsync()
-    {
-        await InvokeVoidAsync("hide", Id);
-        await OnConfirmClick();
-    }
 }