From f51bbcf70b4b8741737d3952f71a88e4bca24d1b Mon Sep 17 00:00:00 2001
From: Ryan Luu <ryan.luu@ryanluu.dev>
Date: Wed, 8 Jan 2025 18:42:38 -0800
Subject: [PATCH] Simplify checkbox logic

---
 FluentAutoClicker/MainPage.xaml    |  6 ++----
 FluentAutoClicker/MainPage.xaml.cs | 30 +++++++++++-------------------
 2 files changed, 13 insertions(+), 23 deletions(-)

diff --git a/FluentAutoClicker/MainPage.xaml b/FluentAutoClicker/MainPage.xaml
index 1a56715..4cb3a57 100644
--- a/FluentAutoClicker/MainPage.xaml
+++ b/FluentAutoClicker/MainPage.xaml
@@ -102,8 +102,7 @@
             <CheckBox
                 x:Name="ClickRepeatCheckBox"
                 x:Uid="ClickRepeatCheckBox"
-                Checked="ClickRepeatCheckBox_Checked"
-                Unchecked="ClickRepeatCheckBox_Unchecked" />
+                Click="CheckBox_Click" />
             <NumberBox
                 x:Name="ClickRepeatAmount"
                 HorizontalAlignment="Left"
@@ -121,8 +120,7 @@
             <CheckBox
                 x:Name="ClickOffsetCheckBox"
                 x:Uid="ClickOffsetCheckBox"
-                Checked="ClickOffsetCheckBox_Checked"
-                Unchecked="ClickOffsetCheckBox_Unchecked" />
+                Click="CheckBox_Click" />
             <NumberBox
                 x:Name="ClickOffsetAmount"
                 HorizontalAlignment="Left"
diff --git a/FluentAutoClicker/MainPage.xaml.cs b/FluentAutoClicker/MainPage.xaml.cs
index b644861..b7e7933 100644
--- a/FluentAutoClicker/MainPage.xaml.cs
+++ b/FluentAutoClicker/MainPage.xaml.cs
@@ -110,7 +110,7 @@ private void SetControlsEnabled(bool isEnabled)
         HotkeyTextBlock.Foreground = Application.Current.Resources[brushKey] as Brush;
     }
 
-    private int GetNumberBoxValue(NumberBox numberBox, int defaultValue)
+    private static int GetNumberBoxValue(NumberBox numberBox, int defaultValue)
     {
         if (!int.TryParse(numberBox.Value.ToString(CultureInfo.InvariantCulture), out int value))
         {
@@ -168,23 +168,15 @@ private void StartToggleButton_OnUnchecked(object sender, RoutedEventArgs e)
         SetControlsEnabled(true);
     }
 
-    private void ClickRepeatCheckBox_Unchecked(object sender, RoutedEventArgs e)
+    private void CheckBox_Click(object sender, RoutedEventArgs e)
     {
-        ClickRepeatAmount.IsEnabled = false;
-    }
-
-    private void ClickRepeatCheckBox_Checked(object sender, RoutedEventArgs e)
-    {
-        ClickRepeatAmount.IsEnabled = true;
-    }
-
-    private void ClickOffsetCheckBox_Unchecked(object sender, RoutedEventArgs e)
-    {
-        ClickOffsetAmount.IsEnabled = false;
-    }
-
-    private void ClickOffsetCheckBox_Checked(object sender, RoutedEventArgs e)
-    {
-        ClickOffsetAmount.IsEnabled = true;
+        if (sender.Equals(ClickRepeatCheckBox))
+        {
+            ClickRepeatAmount.IsEnabled = ClickRepeatCheckBox.IsChecked == true;
+        }
+        else if (sender.Equals(ClickOffsetCheckBox))
+        {
+            ClickOffsetAmount.IsEnabled = ClickOffsetCheckBox.IsChecked == true;
+        }
     }
-}
\ No newline at end of file
+}