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

Bug 1539507 - Fix targeting of button click when button element isn't clicked.

Browse source
This commit is contained in:
Jonathan Kingston 2019-05-01 09:40:48 +01:00
parent 17b2d8c773
commit 3ae1803420
1 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -20,14 +20,21 @@ async function load() {
document.getElementById("redirect-form").addEventListener("submit", (e) => { document.getElementById("redirect-form").addEventListener("submit", (e) => {
e.preventDefault(); e.preventDefault();
const buttonTarget = e.explicitOriginalTarget; let button = "confirm"; // Confirm is the form default.
switch (buttonTarget.id) { let buttonTarget = e.explicitOriginalTarget;
case "confirm": if (buttonTarget.tagName !== "BUTTON") {
confirmSubmit(redirectUrl, cookieStoreId); buttonTarget = buttonTarget.closest("button");
break; }
if (buttonTarget && buttonTarget.id) {
button = buttonTarget.id;
}
switch (button) {
case "deny": case "deny":
denySubmit(redirectUrl); denySubmit(redirectUrl);
break; break;
case "confirm":
confirmSubmit(redirectUrl, cookieStoreId);
break;
} }
}); });
} }