From 377031c0a9696d6072afd8bf6fa5036802108dc5 Mon Sep 17 00:00:00 2001
From: Klesti Selimaj <klesti.selimay@icloud.com>
Date: Sun, 22 Feb 2026 03:20:16 +0100
Subject: [PATCH] Fixed click bugs

---
 .../ui2/components/SaturnModule.java          | 48 ++++++++++++-------
 .../ui2/components/inputs/Toggle.java         |  2 +-
 .../org/saturnclient/ui2/elements/Scroll.java |  2 +-
 3 files changed, 32 insertions(+), 20 deletions(-)

diff --git a/src/main/java/org/saturnclient/ui2/components/SaturnModule.java b/src/main/java/org/saturnclient/ui2/components/SaturnModule.java
index 87da97d..21c8b1c 100644
--- a/src/main/java/org/saturnclient/ui2/components/SaturnModule.java
+++ b/src/main/java/org/saturnclient/ui2/components/SaturnModule.java
@@ -17,6 +17,10 @@ public class SaturnModule extends Element {
 
     private final Module mod;
 
+    static int p = 10; // padding
+    static int s = 14; // settings icon size
+    static int expand = 5; // settings icon click area
+
     public SaturnModule(Module mod) {
         this.mod = mod;
         this.width = 160;
@@ -25,18 +29,17 @@ public class SaturnModule extends Element {
 
     @Override
     public void render(RenderScope renderScope, ElementContext ctx) {
-        int p = 10; // padding
-        int s = 14; // settings icon size
-
         boolean hovered = ctx.isHovering();
         boolean enabled = mod.isEnabled();
 
-        boolean settingsHover = Utils.isHovering(
-                ctx.mouseX - (width - s - p - 5),
-                ctx.mouseY - (height - s - p - 5),
-                s + 10,
-                s + 10,
-                1.0f);
+        int iconX = width - s - p;
+        int iconY = height - s - p;
+
+        boolean settingsHover = ctx.isHovering(
+                getSettingsX(),
+                getSettingsY(),
+                getSettingsHitBox(),
+                getSettingsHitBox());
 
         renderScope.setRenderLayer(RenderLayer::getGuiTextured);
 
@@ -125,8 +128,8 @@ public class SaturnModule extends Element {
         // Settings icon
         renderScope.drawTexture(
                 Textures.SETTINGS,
-                width - s - p,
-                height - s - p,
+                iconX,
+                iconY,
                 0, 0,
                 s, s,
                 settingsColor);
@@ -134,14 +137,11 @@ public class SaturnModule extends Element {
 
     @Override
     public void click(int mouseX, int mouseY) {
-        int p = 10;
-        int s = 14;
-
         boolean settingsHover = Utils.isHovering(
-                mouseX - (width - s - p - 5),
-                mouseY - (height - s - p - 5),
-                s + 10,
-                s + 10,
+                mouseX - getSettingsX(),
+                mouseY - getSettingsY(),
+                getSettingsHitBox(),
+                getSettingsHitBox(),
                 1.0f);
 
         if (settingsHover) {
@@ -150,4 +150,16 @@ public class SaturnModule extends Element {
             mod.setEnabled(!mod.isEnabled());
         }
     }
+
+    private int getSettingsX() {
+        return width - s - p - expand;
+    }
+
+    private int getSettingsY() {
+        return height - s - p - expand;
+    }
+
+    private int getSettingsHitBox() {
+        return s + expand * 2;
+    }
 }
\ No newline at end of file
diff --git a/src/main/java/org/saturnclient/ui2/components/inputs/Toggle.java b/src/main/java/org/saturnclient/ui2/components/inputs/Toggle.java
index c0cfcf6..e692ad9 100644
--- a/src/main/java/org/saturnclient/ui2/components/inputs/Toggle.java
+++ b/src/main/java/org/saturnclient/ui2/components/inputs/Toggle.java
@@ -18,7 +18,7 @@ public class Toggle extends Element {
     @Override
     public void render(RenderScope renderScope, ElementContext ctx) {
         renderScope.drawRoundedRectangle(0, 0, width, height, height,
-                prop.value ? Theme.ACCENT.value : Theme.PRIMARY.value);
+                Theme.withAlpha(0.5f, prop.value ? Theme.ACCENT.value : Theme.PRIMARY.value));
         renderScope.drawRoundedRectangle(prop.value ? 30 : 0, 0, 30, 30, 30,
                 prop.value ? Theme.ACCENT.value : Theme.PRIMARY.value);
     }
diff --git a/src/main/java/org/saturnclient/ui2/elements/Scroll.java b/src/main/java/org/saturnclient/ui2/elements/Scroll.java
index b39616a..68bde0b 100644
--- a/src/main/java/org/saturnclient/ui2/elements/Scroll.java
+++ b/src/main/java/org/saturnclient/ui2/elements/Scroll.java
@@ -115,7 +115,7 @@ public class Scroll extends Element {
 
     @Override
     public void mouseClicked(double mouseX, double mouseY, int button) {
-        ElementRenderer.mouseClicked(children, mouseX, mouseY, button);
+        ElementRenderer.mouseClicked(children, mouseX - padding, mouseY - padding + scroll, button);
     }
 
     @Override