From 742e80015591e92894b5f2c777c44871b1a31072 Mon Sep 17 00:00:00 2001
From: Vladimir Mandic <mandic00@live.com>
Date: Tue, 27 Oct 2020 14:19:24 -0400
Subject: [PATCH] Revert "updated menu handler"

This reverts commit 476a1cc4d42f26d567630bb00f3de2c9eda1688c.
---
 demo/browser.js | 18 +++++++++---------
 demo/menu.js    |  9 ++-------
 2 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/demo/browser.js b/demo/browser.js
index 235e9ebd..7b99587b 100644
--- a/demo/browser.js
+++ b/demo/browser.js
@@ -321,15 +321,15 @@ function setupMenu() {
   menuFX.addHTML('<hr style="min-width: 200px; border-style: inset; border-color: dimgray">');
   menuFX.addLabel('Image Filters');
   menuFX.addBool('Enabled', config.filter, 'enabled');
-  menuFX.addRange('Image width', config.filter, 'width', 100, 3840, 10);
-  menuFX.addRange('Image height', config.filter, 'height', 100, 2160, 10);
-  menuFX.addRange('Brightness', config.filter, 'brightness', -1.0, 1.0, 0.05);
-  menuFX.addRange('Contrast', config.filter, 'contrast', -1.0, 1.0, 0.05);
-  menuFX.addRange('Sharpness', config.filter, 'sharpness', 0, 1.0, 0.05);
-  menuFX.addRange('Blur', config.filter, 'blur', 0, 20, 1);
-  menuFX.addRange('Saturation', config.filter, 'saturation', -1.0, 1.0, 0.05);
-  menuFX.addRange('Hue', config.filter, 'hue', 0, 360, 5);
-  menuFX.addRange('Pixelate', config.filter, 'pixelate', 0, 32, 1);
+  menuFX.addRange('Image width', config.filter, 'width', 100, 3840, 10, (val) => config.filter.width = parseInt(val));
+  menuFX.addRange('Image height', config.filter, 'height', 100, 2160, 10, (val) => config.filter.height = parseInt(val));
+  menuFX.addRange('Brightness', config.filter, 'brightness', -1.0, 1.0, 0.05, (val) => config.filter.brightness = parseFloat(val));
+  menuFX.addRange('Contrast', config.filter, 'contrast', -1.0, 1.0, 0.05, (val) => config.filter.contrast = parseFloat(val));
+  menuFX.addRange('Sharpness', config.filter, 'sharpness', 0, 1.0, 0.05, (val) => config.filter.sharpness = parseFloat(val));
+  menuFX.addRange('Blur', config.filter, 'blur', 0, 20, 1, (val) => config.filter.blur = parseInt(val));
+  menuFX.addRange('Saturation', config.filter, 'saturation', -1.0, 1.0, 0.05, (val) => config.filter.saturation = parseFloat(val));
+  menuFX.addRange('Hue', config.filter, 'hue', 0, 360, 5, (val) => config.filter.hue = parseInt(val));
+  menuFX.addRange('Pixelate', config.filter, 'pixelate', 0, 32, 1, (val) => config.filter.pixelate = parseInt(val));
   menuFX.addBool('Negative', config.filter, 'negative');
   menuFX.addBool('Sepia', config.filter, 'sepia');
   menuFX.addBool('Vintage', config.filter, 'vintage');
diff --git a/demo/menu.js b/demo/menu.js
index 014521d9..493efc06 100644
--- a/demo/menu.js
+++ b/demo/menu.js
@@ -133,15 +133,10 @@ class Menu {
   async addRange(title, object, variable, min, max, step, callback) {
     const el = document.createElement('div');
     el.className = 'menu-item';
-    const arr = Array.isArray(variable);
-    el.innerHTML = `<input class="menu-range" type="range" id="${this.newID}" min=${min} max=${max} step=${step} value=${arr ? object[variable[0]] : object[variable]}>${title}`;
+    el.innerHTML = `<input class="menu-range" type="range" id="${this.newID}" min="${min}" max="${max}" step="${step}" value="${object[variable]}">${title}`;
     this.container.appendChild(el);
     el.addEventListener('change', (evt) => {
-      const int = parseInt(evt.target.value) === parseFloat(evt.target.value);
-      const val = Array.isArray(variable) ? variable : [variable];
-      for (const item of val) {
-        object[item] = int ? parseInt(evt.target.value) : parseFloat(evt.target.value);
-      }
+      object[variable] = evt.target.value;
       evt.target.setAttribute('value', evt.target.value);
       if (callback) callback(evt.target.value);
     });