updateConfig when move sliders too (#145)

onchange only would trigger updateConfig when user releases the mouse from sliders. Adding oninput also allows updateConfig when dragging the sliders, which provides immediate feedback of the changes for the user.
easytarget · Sep 24, 2021 · a536b94 · a536b94
1 parent 4d253bf
commit a536b94
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/index_other.h b/index_other.h
@@ -258,6 +258,11 @@ const uint8_t index_simple_html[] = R"=====(<!doctype html>
       .forEach(el => {
         el.onchange = () => updateConfig(el)
       })
+    document
+      .querySelectorAll('input[type="range"]')
+      .forEach(el => {
+        el.oninput = () => updateConfig(el)
+      })
 
     // Custom actions
     // Detection and framesize

diff --git a/index_ov2640.h b/index_ov2640.h
@@ -498,6 +498,11 @@ const uint8_t index_ov2640_html[] = R"=====(<!doctype html>
       .forEach(el => {
         el.onchange = () => updateConfig(el)
       })
+    document
+      .querySelectorAll('input[type="range"]')
+      .forEach(el => {
+        el.oninput = () => updateConfig(el)
+      })
 
     // Custom actions
     // Gain

diff --git a/index_ov3660.h b/index_ov3660.h
@@ -510,6 +510,11 @@ const uint8_t index_ov3660_html[] = R"=====(<!doctype html>
       .forEach(el => {
         el.onchange = () => updateConfig(el)
       })
+    document
+      .querySelectorAll('input[type="range"]')
+      .forEach(el => {
+        el.oninput = () => updateConfig(el)
+      })
 
     // Custom actions
     // Gain