From 8743cba65a712f93b575ddf1c83d38cda58abc46 Mon Sep 17 00:00:00 2001
From: Carter <carter@adaptiveagriculture.ca>
Date: Tue, 25 Feb 2025 11:25:10 -0600
Subject: [PATCH] onClick is on menu item instead of the checkbox itself

---
 src/common/ResponsiveTable.tsx | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/common/ResponsiveTable.tsx b/src/common/ResponsiveTable.tsx
index c45d3ba..b1332e9 100644
--- a/src/common/ResponsiveTable.tsx
+++ b/src/common/ResponsiveTable.tsx
@@ -497,11 +497,16 @@ export default function ResponsiveTable<T extends Object>(props: Props<T>) {
         disableAutoFocusItem>
           {columns?.map((column, index) => {
             return (
-              <MenuItem key={"column-list-"+index} aria-label="Open User Settings" dense sx={{ marginLeft: -0.75 }}>
+              <MenuItem 
+                onClick={() => filter(column.title)}
+                key={"column-list-"+index} 
+                aria-label="Open User Settings" 
+                dense 
+                sx={{ marginLeft: -0.75 }}
+              >
                 <ListItemIcon>
                   <Checkbox 
                     checked={!filterList.includes(column.title)}
-                    onClick={() => filter(column.title)}
                   />
                 </ListItemIcon>
                 <ListItemText primary={column.title} />