From fde32bac22d7f25865783000d42e40fc4e655363 Mon Sep 17 00:00:00 2001 From: stoically Date: Fri, 1 Mar 2019 13:30:10 +0100 Subject: [PATCH] Avoid using explicitOriginalTarget on confirm-page #1331 --- src/js/confirm-page.js | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/src/js/confirm-page.js b/src/js/confirm-page.js index fef08d4..5895342 100644 --- a/src/js/confirm-page.js +++ b/src/js/confirm-page.js @@ -18,17 +18,13 @@ async function load() { document.getElementById("current-container-name").textContent = currentContainer.name; } - document.getElementById("redirect-form").addEventListener("submit", (e) => { + document.getElementById("confirm").addEventListener("click", (e) => { e.preventDefault(); - const buttonTarget = e.explicitOriginalTarget; - switch (buttonTarget.id) { - case "confirm": - confirmSubmit(redirectUrl, cookieStoreId); - break; - case "deny": - denySubmit(redirectUrl); - break; - } + confirmSubmit(redirectUrl, cookieStoreId); + }); + document.getElementById("deny").addEventListener("click", (e) => { + e.preventDefault(); + denySubmit(redirectUrl); }); }