rowan/multi-account-containers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Avoid using explicitOriginalTarget on confirm-page #1331

Browse source
This commit is contained in:
stoically 2019-03-01 13:30:10 +01:00
parent 6f4d3c4327
commit fde32bac22
No known key found for this signature in database
GPG key ID: 3026ED2F90EE0DE9
1 changed files with 6 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/js/confirm-page.js
View file

@ -18,17 +18,13 @@ async function load() {
document.getElementById("current-container-name").textContent = currentContainer.name; document.getElementById("current-container-name").textContent = currentContainer.name;
} }
document.getElementById("redirect-form").addEventListener("submit", (e) => { document.getElementById("confirm").addEventListener("click", (e) => {
e.preventDefault(); e.preventDefault();
const buttonTarget = e.explicitOriginalTarget; confirmSubmit(redirectUrl, cookieStoreId);
switch (buttonTarget.id) { });
case "confirm": document.getElementById("deny").addEventListener("click", (e) => {
confirmSubmit(redirectUrl, cookieStoreId); e.preventDefault();
break; denySubmit(redirectUrl);
case "deny":
denySubmit(redirectUrl);
break;
}
}); });
} }