Revert "fix: simplify asset copy button behavior by removing some logic"

This reverts commit 09154ef.

Signed-off-by: F.N. Claessen <[email protected]>

Author: Flix6x

flexmeasures/ui/static/js/ui-utils.js

@@ -376,6 +376,7 @@ export function initCopyAssetButtons() {
         const targetAccountId = btn.dataset.targetAccountId || null;
 
         btn.addEventListener("click", function (event) {
+            const openInNewTab = event.ctrlKey || event.metaKey;
             const url = targetAccountId
                 ? "/api/v3_0/assets/" + assetId + "/copy?account=" + targetAccountId
                 : "/api/v3_0/assets/" + assetId + "/copy";
@@ -387,7 +388,11 @@ export function initCopyAssetButtons() {
                     showToast("Asset copied successfully.", "success");
                     setTimeout(() => {
                         const dest = "/assets/" + data.asset + "/properties";
-                        window.open(dest, "_blank");
+                        if (openInNewTab) {
+                            window.open(dest, "_blank");
+                        } else {
+                            window.location.href = dest;
+                        }
                     }, 1500);
                 }),
                 "Failed to copy asset"