Merge pull request #123 from CSSLab/copilot/fix-122

Fix: Automatically close game selection popup on mobile when game is selected

Author: kevinjosethomas

src/components/Analysis/AnalysisGameList.tsx

@@ -51,6 +51,7 @@ interface AnalysisGameListProps {
     setCurrentMove?: Dispatch<SetStateAction<number>>,
   ) => Promise<void>
   onCustomAnalysis?: () => void
+  onGameSelected?: () => void // Called when a game is selected (for mobile popup closing)
   refreshTrigger?: number // Used to trigger refresh when custom analysis is added
 }
 
@@ -61,6 +62,7 @@ export const AnalysisGameList: React.FC<AnalysisGameListProps> = ({
   loadNewUserGames,
   loadNewCustomGame,
   onCustomAnalysis,
+  onGameSelected,
   refreshTrigger,
 }) => {
   const router = useRouter()
@@ -559,6 +561,8 @@ export const AnalysisGameList: React.FC<AnalysisGameListProps> = ({
                             } else {
                               router.push(`/analysis/${game.id}/${game.type}`)
                             }
+                            // Call the callback if provided (for mobile popup closing)
+                            onGameSelected?.()
                           }}
                           className="flex flex-1 cursor-pointer items-center justify-between overflow-hidden py-1"
                         >

src/pages/analysis/[...id].tsx

@@ -905,6 +905,7 @@ const Analysis: React.FC<Props> = ({
                   setShowCustomModal(true)
                   setShowGameListMobile(false)
                 }}
+                onGameSelected={() => setShowGameListMobile(false)}
                 refreshTrigger={refreshTrigger}
               />
             </div>