fix: use children.length instead of querySelectorAll #336

ArgoZhang · densen2014 · web-flow · commit 2c4fc54041ec · 2023-01-19T12:05:30.000+08:00
Co-authored-by: alex chow &lt;zhouchuanglin@gmail.com&gt;
diff --git a/src/BootstrapBlazor/wwwroot/modules/modal.js b/src/BootstrapBlazor/wwwroot/modules/modal.js
@@ -161,8 +161,7 @@ export class Modal extends BlazorComponent {
     }
 
     _hide() {
-        const dialogs = this._element.querySelectorAll('.modal-dialog')
-        if (dialogs.length === 1) {
+        if (this._element.children.length === 1) {
             this._modal.hide()
         } else {
             this._invoker.invokeMethodAsync(this._invokerCloseMethod)

Original file line number	Diff line number	Diff line change
`@@ -161,8 +161,7 @@ export class Modal extends BlazorComponent {`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`_hide() {`
`164`		`- const dialogs = this._element.querySelectorAll('.modal-dialog')`
`165`		`- if (dialogs.length === 1) {`
	`164`	`+ if (this._element.children.length === 1) {`
`166`	`165`	`this._modal.hide()`
`167`	`166`	`} else {`
`168`	`167`	`this._invoker.invokeMethodAsync(this._invokerCloseMethod)`