From 55aa2ddb821c90b3d3ec2d251b68fd30cba7178a Mon Sep 17 00:00:00 2001 From: Daniel Eisterhold Date: Fri, 23 Jan 2026 17:52:46 -0600 Subject: [PATCH] Refactor showBusy method for loader class handling Refactor showBusy method to ensure loader class management is handled correctly. --- js/workflows/workflow.js | 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/js/workflows/workflow.js b/js/workflows/workflow.js index 33c7ada..76d9c7d 100644 --- a/js/workflows/workflow.js +++ b/js/workflows/workflow.js @@ -150,19 +150,23 @@ class Workflow { } async showBusy(functionPromise, darkBackground = true) { - if (this.loader) { - if (darkBackground) { - this.loader.classList.add("overlay"); - } else { - this.loader.classList.remove("overlay"); + try { + if (this.loader) { + if (darkBackground) { + this.loader.classList.add("overlay"); + } else { + this.loader.classList.remove("overlay"); + } + this.loader.classList.add("busy"); + } + let result = await functionPromise; + + return result; + } finally { + if (this.loader) { + this.loader.classList.remove("busy"); } - this.loader.classList.add("busy"); - } - let result = await functionPromise; - if (this.loader) { - this.loader.classList.remove("busy"); } - return result; } async parseParams(urlParams) {