From 6b21112d5994a8ef808ab8179eb5f13bb2cdaf41 Mon Sep 17 00:00:00 2001 From: Suzu008 Date: Tue, 24 Mar 2026 08:31:20 +0800 Subject: [PATCH] refactor: use replaceChildren() instead of while-loop DOM cleanup --- ts/cryptioSaveVideo.ts | 4 +--- ts/cryptioUploadVideo.ts | 8 ++------ 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/ts/cryptioSaveVideo.ts b/ts/cryptioSaveVideo.ts index 770aae1..1e59ba4 100644 --- a/ts/cryptioSaveVideo.ts +++ b/ts/cryptioSaveVideo.ts @@ -125,9 +125,7 @@ async function updateVideoWidget(widget: any, videoInfo: any) { widget.element.style.overflow = "hidden"; // Clear previous content - while (widget.element.firstChild) { - widget.element.removeChild(widget.element.firstChild); - } + widget.element.replaceChildren(); // Add new video widget.element.appendChild(video); diff --git a/ts/cryptioUploadVideo.ts b/ts/cryptioUploadVideo.ts index 8b7bc80..7e11478 100644 --- a/ts/cryptioUploadVideo.ts +++ b/ts/cryptioUploadVideo.ts @@ -125,9 +125,7 @@ async function updateVideoWidget(widget: any, filename: string) { widget.element.style.overflow = "hidden"; // Clear previous content - while (widget.element.firstChild) { - widget.element.removeChild(widget.element.firstChild); - } + widget.element.replaceChildren(); // Add new video widget.element.appendChild(video); @@ -159,9 +157,7 @@ function clearVideoWidget(widget: any) { widget.element.hidden = true; widget.value = { hidden: true }; - while (widget.element.firstChild) { - widget.element.removeChild(widget.element.firstChild); - } + widget.element.replaceChildren(); app.rootGraph?.setDirtyCanvas(true, false); }