Merge pull request #2060 from harshalranjhani/harshalranjhani/2053/op…

…en-goal-modal-with-spacebar-key

feat: open goal modal with spacebar key

src/hooks/useGoalSelection.ts

@@ -14,6 +14,7 @@ export const useGoalSelection = (goals: GoalItem[]): GoalItem | undefined => {
   const downPress = useKeyPress("ArrowDown");
   const rightPress = useKeyPress("ArrowRight");
   const leftPress = useKeyPress("ArrowLeft");
+  const spacePress = useKeyPress("Space");
 
   const focusIndex = Number(searchParams.get("focus") || -1);
 
@@ -62,6 +63,15 @@ export const useGoalSelection = (goals: GoalItem[]): GoalItem | undefined => {
     [goals],
   );
 
+  const openGoalActionsModal = (goal: GoalItem) => {
+    if (!goal) return;
+    const newPath = goal.parentGoalId === "root" ? `/goals/root/${goal.id}` : `/goals/${goal.parentGoalId}/${goal.id}`;
+
+    navigate(`${newPath}?showOptions=true`, {
+      state: location.state,
+    });
+  };
+
   useEffect(() => {
     const handleKeyDown = (e: KeyboardEvent) => {
       if (["ArrowUp", "ArrowDown", "ArrowLeft", "ArrowRight"].includes(e.key)) {
@@ -90,10 +100,14 @@ export const useGoalSelection = (goals: GoalItem[]): GoalItem | undefined => {
       window.history.back();
     }
 
+    if (spacePress && goals.length > 0) {
+      openGoalActionsModal(goals[focusIndex]);
+    }
+
     return () => {
       window.removeEventListener("keydown", handleKeyDown);
     };
-  }, [downPress, upPress, rightPress, leftPress]);
+  }, [downPress, upPress, rightPress, leftPress, spacePress]);
 
   return goals[focusIndex];
 };

src/hooks/useKeyPress.ts

@@ -6,7 +6,7 @@ export const useKeyPress = (targetKey: string) => {
   const keyHandler = useCallback(
     (event: KeyboardEvent) => {
       event.stopPropagation();
-      if (event.key === targetKey) {
+      if (event.code === targetKey) {
         setKeyPressed(event.type === "keydown");
       }
     },