From ce050d2736bc9e8492cfe8bd66b2514a5e44f6f7 Mon Sep 17 00:00:00 2001
From: RenanMsV <r.delaneza@gmail.com>
Date: Tue, 20 Apr 2021 13:26:21 -0300
Subject: [PATCH] Fixes nathanp/crypto-price-widget#67

---
 css/app.css      |  2 +-
 index.html       |  8 ++++++++
 js/app_common.js | 32 ++++++++++++++++++++++++++++++--
 3 files changed, 39 insertions(+), 3 deletions(-)

diff --git a/css/app.css b/css/app.css
index ec1fd0d..5cc24ae 100644
--- a/css/app.css
+++ b/css/app.css
@@ -171,7 +171,7 @@ ul {
       .coin-list li span.draggable {
         width: 100%;
       }
-      .coin-list input[type="number"] {
+      .coin-list input[type="number"], .update-interval {
         border: none;
         padding: 5px;
         background: rgba(0, 0, 0, 0);
diff --git a/index.html b/index.html
index 3e4a477..dc4c3b0 100644
--- a/index.html
+++ b/index.html
@@ -69,6 +69,14 @@ <h3>Choose Your Base Currency</h3>
       </label>
       </div>
 
+      <div class="checkbox-wrapper">
+        <input id="update-interval" class="update-interval" name="update-interval" min="5" max="1000" step="1" value="5" type="number">
+        <label class="">
+          Update interval in seconds
+        <div></div>
+      </label>
+      </div>
+
       <h3>Tip Jar</h3>
       <ul id="tips">
         <li>BTC: 17iENfaJkEpxGXW7mgdFh9hGMZV65R2zVL</li>
diff --git a/js/app_common.js b/js/app_common.js
index 3efbc7c..86657b6 100644
--- a/js/app_common.js
+++ b/js/app_common.js
@@ -6,6 +6,17 @@ const remote = require("electron").remote;
 //user settings
 const settings = require("electron-settings");
 
+// update interval
+var appRefresh; // refresh timer handle
+var update_interval = 5;
+var update_interval_element = document.getElementById("update-interval")
+if (settings.has("user.updateInterval")) {
+  update_interval = settings.get("user.updateInterval");
+  update_interval_element.value = update_interval;
+} else {
+  settings.set("user.updateInterval", update_interval); // 5 seconds by default
+}
+
 //default coins
 if (settings.has("user.coins")) {
   //do nothing because coins already set
@@ -311,10 +322,15 @@ function updateData() {
       }); //response.json().then
     } //function(response)
   ); //then
+  refreshUpdateInterval()
+} //updateData()
+
+function refreshUpdateInterval () { // clears timeout and reload prices page
+  clearTimeout(appRefresh); // clears older timeout
   appRefresh = setTimeout(function () {
     updateData();
-  }, 5000); // run this once every 5 seconds
-} //updateData()
+  }, update_interval * 1000); // run this once every user assigned seconds
+}
 
 // Let's do this thing!
 initData();
@@ -474,6 +490,18 @@ pinCheck.onclick = function (event) {
   }
 };
 
+/***********
+ * UPDATE INTERVAL
+ *************/
+update_interval_element.onchange = function (event) { // saves user update interval
+  let input = event.target;
+  input.value = Math.round(input.value);
+  input.value = Math.max(5, Math.min(input.value, 1000)); // clamps between 5 and 1000
+  settings.set("user.updateInterval", input.value);
+  update_interval = input.value;
+  refreshUpdateInterval()
+}
+
 /*******
  * APP UI
  ********/