From cd38ec8c1081ad30664d2862dc3b9ca6888b05d2 Mon Sep 17 00:00:00 2001
From: matt1432 <matt@nelim.org>
Date: Tue, 12 Sep 2023 16:44:47 -0400
Subject: [PATCH] perf(ags): add throttle to brightness slider

---
 config/ags/js/quick-settings/slider-box.js | 39 ++++++++++++++++++----
 1 file changed, 32 insertions(+), 7 deletions(-)

diff --git a/config/ags/js/quick-settings/slider-box.js b/config/ags/js/quick-settings/slider-box.js
index d926c42..63c635d 100644
--- a/config/ags/js/quick-settings/slider-box.js
+++ b/config/ags/js/quick-settings/slider-box.js
@@ -1,4 +1,4 @@
-const { Box, Slider, Label, Icon } = ags.Widget;
+const { Box, Slider, Label, Icon, EventBox } = ags.Widget;
 const { Audio } = ags.Service;
 const { exec } = ags.Utils;
 
@@ -10,6 +10,18 @@ const items = {
   0: 'audio-volume-muted-symbolic',
 };
 
+let throttleTimer;
+
+const throttle = (callback, time) => {
+  if (throttleTimer) return;
+    throttleTimer = true;
+    setTimeout(() => {
+        callback();
+        throttleTimer = false;
+    }, time);
+}
+
+
 export const SliderBox = Box({
   className: 'slider-box',
   vertical: true,
@@ -64,12 +76,25 @@ export const SliderBox = Box({
           icon: 'display-brightness-symbolic',
         }),
 
-        Slider({
-          value: `${exec('brightnessctl get') / 2.55}`,
-          onChange: "brightnessctl set {}%",
-          min: 0,
-          max: 100,
-          draw_value: false,
+        EventBox({
+          onHover: box => box.child._canChange = false,
+          onHoverLost: box => box.child._canChange = true,
+          child: Slider({
+            properties: [
+              ['canChange', true],
+            ],
+            onChange: ({ value }) => {
+              throttle(() => exec('brightnessctl set ' + value + ' %'), 20);
+            },
+            connections: [[1000, slider => {
+              if (slider._canChange) {
+                slider.value = exec('brightnessctl get');
+              }
+            }]],
+            min: 0,
+            max: 255,
+            draw_value: false,
+          }),
         }),
       ],
     }),