From 3ef2bce9f04528ea6bdf935022f410fdc4b7ac23 Mon Sep 17 00:00:00 2001 From: Victor Seiji Hariki Date: Tue, 13 Dec 2022 11:24:22 -0300 Subject: [PATCH] fixed loading of boolean values from localStorage It seems we were doing localStorage wrong for boolean values. It just worked for enable_hr by chance. Now it should work properly. Signed-off-by: Victor Seiji Hariki --- js/index.js | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/js/index.js b/js/index.js index 5426fc9..8bb9f1b 100644 --- a/js/index.js +++ b/js/index.js @@ -904,15 +904,20 @@ function loadSettings() { : localStorage.getItem("mask_blur"); var _seed = localStorage.getItem("seed") == null ? -1 : localStorage.getItem("seed"); - var _enable_hr = Boolean( - localStorage.getItem("enable_hr") == (null || "false") + + let _enable_hr = + localStorage.getItem("enable_hr") === null ? false - : localStorage.getItem("enable_hr") - ); - var _sync_cursor_size = Boolean( - localStorage.getItem("sync_cursor_size") == (null || "true") + : localStorage.getItem("enable_hr") === "true"; + + let _sync_cursor_size = + localStorage.getItem("sync_cursor_size") === null ? true - : localStorage.getItem("sync_cursor_size") + : localStorage.getItem("sync_cursor_size") === "true"; + + console.debug( + localStorage.getItem("sync_cursor_size"), + Boolean(localStorage.getItem("sync_cursor_size")) ); // set the values into the UI