refactor: use existence instead of boolean string in local storage

Umami needs `umami.disabled` value to be `1` to actually disable it. Also, local storage can't store booleans, hence 'true' and 'false' are stored as strings, which is dumb. Just remove the item when it needs to be set false.

1 files changed, 4 insertions, 1 deletions

diff --git a/assets/js/settings.js b/assets/js/settings.js
index 053786d6..52675143 100644
--- a/assets/js/settings.js
+++ b/assets/js/settings.js
@@ -96,7 +96,10 @@
 
           // add event listener to checkbox
           element.addEventListener('change', () => {
-            localStorage.setItem(checkboxOption.storage, element.checked);
+            if (element.checked)
+              localStorage.setItem(checkboxOption.storage, 1);
+            else
+              localStorage.removeItem(checkboxOption.storage);
           });
 
           // append checkbox and label to container