From 0ce11b6908ce85c396b39c2bc67ce2cf46e85d7f Mon Sep 17 00:00:00 2001
From: Ronit Panda <72537293+rtpa25@users.noreply.github.com>
Date: Mon, 25 Sep 2023 15:55:03 +0530
Subject: [PATCH] fix: adds cursor pointer to table column header (#1715)

* fix: adds cursro pointer to table column header

* fix: lifts up the onclick handler to header not tag making consistent with other tables
---
 front/src/modules/ui/board/components/BoardColumn.tsx     | 4 ++--
 .../src/modules/ui/table/components/EntityTableHeader.tsx | 8 ++++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/front/src/modules/ui/board/components/BoardColumn.tsx b/front/src/modules/ui/board/components/BoardColumn.tsx
index c15deab73..915cb3b47 100644
--- a/front/src/modules/ui/board/components/BoardColumn.tsx
+++ b/front/src/modules/ui/board/components/BoardColumn.tsx
@@ -97,8 +97,8 @@ export const BoardColumn = ({
 
   return (
     <StyledColumn isFirstColumn={isFirstColumn}>
-      <StyledHeader>
-        <Tag onClick={handleTitleClick} color={color ?? 'gray'} text={title} />
+      <StyledHeader onClick={handleTitleClick}>
+        <Tag color={color ?? 'gray'} text={title} />
         {!!totalAmount && <StyledAmount>${totalAmount}</StyledAmount>}
         <StyledNumChildren>{numChildren}</StyledNumChildren>
       </StyledHeader>
diff --git a/front/src/modules/ui/table/components/EntityTableHeader.tsx b/front/src/modules/ui/table/components/EntityTableHeader.tsx
index 4dbd1dd65..040d30196 100644
--- a/front/src/modules/ui/table/components/EntityTableHeader.tsx
+++ b/front/src/modules/ui/table/components/EntityTableHeader.tsx
@@ -72,6 +72,10 @@ const StyledEntityTableColumnMenu = styled(EntityTableColumnMenu)`
   z-index: ${({ theme }) => theme.lastLayerZIndex};
 `;
 
+const StyledTableHead = styled.thead`
+  cursor: pointer;
+`;
+
 export const EntityTableHeader = () => {
   const [resizeFieldOffset, setResizeFieldOffset] = useRecoilState(
     resizeFieldOffsetState,
@@ -155,7 +159,7 @@ export const EntityTableHeader = () => {
   }, []);
 
   return (
-    <thead data-select-disable>
+    <StyledTableHead data-select-disable>
       <tr>
         <th
           style={{
@@ -214,6 +218,6 @@ export const EntityTableHeader = () => {
           )}
         </th>
       </tr>
-    </thead>
+    </StyledTableHead>
   );
 };