diff options
author | Einar Egilsson <einar@einaregilsson.com> | 2020-02-06 08:51:23 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-02-06 08:51:23 +0000 |
commit | 3d52584077280d9d0e2bcee6b1adf9ff2f9bd4d3 (patch) | |
tree | e51f59174a1a6381bdfb28f09c9744a74d788495 /js/editredirect.js | |
parent | 6a12784be0ac4759e92580c01ac8bfd7f91c4f1f (diff) | |
parent | 226071781cb4bc63f4587cf2df5b3c95b5fd0ccd (diff) |
Merge pull request #185 from ncolletti/nav-btns
grouping works on enable toggle. code clean up.
Diffstat (limited to 'js/editredirect.js')
-rw-r--r-- | js/editredirect.js | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/js/editredirect.js b/js/editredirect.js index c770199..ca4ea07 100644 --- a/js/editredirect.js +++ b/js/editredirect.js @@ -29,7 +29,6 @@ function saveRedirect() { REDIRECTS[activeRedirect.index] = savedRedirect; //To strip out any extra crap we've added } else { REDIRECTS.push(savedRedirect); - let newNode = template.cloneNode(true); newNode.removeAttribute('id'); el('.redirect-rows').appendChild(newNode); @@ -59,11 +58,10 @@ function editFormChange() { let prop = input.getAttribute('data-bind'); activeRedirect[prop] = input.value; } - activeRedirect.appliesTo = []; for (let input of el('#apply-to').querySelectorAll('input:checked')) { activeRedirect.appliesTo.push(input.value); - } + } activeRedirect.processMatches = el('#process-matches option:checked').value; activeRedirect.patternType = el('[name="patterntype"]:checked').value; @@ -78,8 +76,8 @@ function editFormChange() { var deleteIndex; function confirmDeleteRedirect(index) { deleteIndex = index; - let redirect = REDIRECTS[deleteIndex]; - showForm('#delete-redirect-form', redirect); + let redirect = REDIRECTS[deleteIndex]; + showForm('#delete-redirect-form', redirect); } function deleteRedirect() { @@ -104,7 +102,7 @@ function setupEditAndDeleteEventListeners() { el('#confirm-delete').addEventListener('click', deleteRedirect); el('#cancel-delete').addEventListener('click', cancelDelete); - el('#advanced-toggle a').addEventListener('click', toggleAdvancedOptions); + el('#advanced-toggle a').addEventListener('click', toggleAdvancedOptions); el('#create-new-redirect').addEventListener('click', createNewRedirect); //Listen to any change from the edit form... |