Fix formatting, add missing semicolons, and change constant names to …

…uppercase
do0ori · Feb 22, 2024 · 0660719 · 0660719
1 parent 59f4ae9
commit 0660719
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 13 deletions.
diff --git a/index.html b/index.html
@@ -14,7 +14,9 @@
         <input type="number" min="1" id="total-input" placeholder="Total" />
     </div>
     <div id="context-menu">
-        <div class="context-menu-item" id="reset-progress">Reset progress to 0</div>
+        <div class="context-menu-item" id="reset-progress">
+            Reset progress to 0
+        </div>
         <div class="context-menu-item">
             <label for="bar-color-picker">Bar color</label>
             <input type="color" id="bar-color-picker">

diff --git a/progressbar.js b/progressbar.js
@@ -1,11 +1,11 @@
-const totalKey = "total";
-const progressKey = "progress";
-const barColorKey = "barColor";
+const TOTAL_KEY = "total";
+const PROGRESS_KEY = "progress";
+const BAR_COLOR_KEY = "barColor";
 
 document.addEventListener("DOMContentLoaded", function () {
-    let total = parseInt(localStorage.getItem(totalKey)) || 1;
-    let progress = parseInt(localStorage.getItem(progressKey)) || 0;
-    let barColor = localStorage.getItem(barColorKey) || "#4CAF50";
+    let total = parseInt(localStorage.getItem(TOTAL_KEY)) || 1;
+    let progress = parseInt(localStorage.getItem(PROGRESS_KEY)) || 0;
+    let barColor = localStorage.getItem(BAR_COLOR_KEY) || "#4CAF50";
     const progressContainer = document.getElementById("progress-container");
     const progressBar = document.getElementById("progress-bar");
     const countDisplay = document.getElementById("count-display");
@@ -34,30 +34,30 @@ document.addEventListener("DOMContentLoaded", function () {
 
         if (!isNaN(newTotal) && newTotal > 0 && newTotal !== total) {
             total = newTotal;
-            localStorage.setItem(totalKey, total);
+            localStorage.setItem(TOTAL_KEY, total);
             totalInput.value = total;
             resetProgress();
         }
     };
 
     const resetBarColor = () => {
         barColor = barColorPicker.value = "#4CAF50";
-        localStorage.setItem(barColorKey, barColor);
+        localStorage.setItem(BAR_COLOR_KEY, barColor);
         updateBarColor();
     };
 
     const decreaseProgress = () => {
         if (progress > 0) {
             progress--;
-            localStorage.setItem(progressKey, progress);
+            localStorage.setItem(PROGRESS_KEY, progress);
             resetProgress();
         }
     };
 
     const increaseProgress = () => {
         if (progress < total) {
             progress++;
-            localStorage.setItem(progressKey, progress);
+            localStorage.setItem(PROGRESS_KEY, progress);
             resetProgress();
         }
     };
@@ -88,7 +88,7 @@ document.addEventListener("DOMContentLoaded", function () {
 
     const handlBarColor = () => {
         barColor = barColorPicker.value;
-        localStorage.setItem(barColorKey, barColor);
+        localStorage.setItem(BAR_COLOR_KEY, barColor);
         updateBarColor();
     };
 
@@ -102,7 +102,7 @@ document.addEventListener("DOMContentLoaded", function () {
     });
     document.getElementById("reset-progress").addEventListener("click", () => {
         progress = 0;
-        localStorage.setItem(progressKey, progress);
+        localStorage.setItem(PROGRESS_KEY, progress);
         resetProgress();
     });
     barColorPicker.addEventListener("change", handlBarColor);