From 291d7ecf72a3ce6ee1a9c6e7eee4f370d1cbf80b Mon Sep 17 00:00:00 2001 From: justusaac Date: Fri, 2 May 2025 19:20:07 -0500 Subject: [PATCH] Move stopPropagation call to top of function Based on PR feedback. --- src/webui/www/private/scripts/dynamicTable.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/webui/www/private/scripts/dynamicTable.js b/src/webui/www/private/scripts/dynamicTable.js index 4305c74c2..74d53e1a1 100644 --- a/src/webui/www/private/scripts/dynamicTable.js +++ b/src/webui/www/private/scripts/dynamicTable.js @@ -2355,6 +2355,7 @@ window.qBittorrent.DynamicTable ??= (() => { checkbox.type = "checkbox"; checkbox.className = "RenamingCB"; checkbox.addEventListener("click", (e) => { + e.stopPropagation(); const targetId = e.target.dataset.id; const ids = []; // when holding shift, set all files between the previously selected one and the clicked one @@ -2388,7 +2389,6 @@ window.qBittorrent.DynamicTable ??= (() => { } that.updateGlobalCheckbox(); that.onRowSelectionChange(that.getNode(targetId)); - e.stopPropagation(); that.prevCheckboxNum = targetId; }); checkbox.indeterminate = false;