updateConfig when move sliders too

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 · Jul 21, 2021 · dc47637 · dc47637
1 parent 3ba9e81
commit dc47637
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/index_other.h b/index_other.h
@@ -254,10 +254,15 @@ const uint8_t index_simple_html[] = R"=====(<!doctype html>
 
     // Attach default on change action
     document
-      .querySelectorAll('.action-setting')
+      .querySelectorAll('.default-action')
       .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