fix: display stone count immediately on board pits

Reviewed-on: #1

apps/frontend/src/components/board/BoardView.tsx

@@ -14,14 +14,25 @@ const BoardView: FunctionComponent<{
   boardViewModel: BoardViewModel;
   revert: boolean;
   onPitSelect: (index: number, pit: Pit) => void;
-}> = ({ game, context, boardId, boardViewModel, revert, onPitSelect: onPitSelect }) => {
+}> = ({ game, context, boardId, boardViewModel, revert, onPitSelect }) => {
   const mancalaGame = game?.mancalaGame;
   const theme = context.themeManager.theme;
 
-  const createPitView = (key: any, pit: Pit, pitViewModel: PitViewModel) => {
+  // Compute pit width based on viewport (each column is 11vw)
-    return <PitView key={key} pitViewModel={pitViewModel} onClick={() => onPitSelect(pit.index, pit)} />;
+  const pitWidth = typeof window !== "undefined" ? window.innerWidth * 0.11 : 100;
-  };
+
-  const createPitViewList = (pits: Pit[]) => pits.map((pit, index) => createPitView(index, pit, boardViewModel.pits[pit.index]));
+  const createPitView = (key: any, pit: Pit, pitViewModel: PitViewModel) => (
+    <PitView
+      key={key}
+      pitViewModel={pitViewModel}
+      context={context}
+      onClick={() => onPitSelect(pit.index, pit)}
+      fontSize={pitWidth / 6}
+    />
+  );
+
+  const createPitViewList = (pits: Pit[]) =>
+    pits.map((pit, index) => createPitView(index, pit, boardViewModel.pits[pit.index]));
 
   const player1Pits = createPitViewList(mancalaGame?.board.player1Pits);
   const player2Pits = createPitViewList(mancalaGame?.board.player2Pits);
@@ -29,6 +40,7 @@ const BoardView: FunctionComponent<{
   const player2BankIndex = mancalaGame?.board.player2BankIndex();
   const player1BankViewModel = boardViewModel.pits[player1BankIndex];
   const player2BankViewModel = boardViewModel.pits[player2BankIndex];
+
   return (
     <div className="board" style={{ background: theme.boardColor }}>
       <StoreView
@@ -36,12 +48,14 @@ const BoardView: FunctionComponent<{
         pitViewModel={revert ? player2BankViewModel : player1BankViewModel}
         gridColumn="8 / 9"
         gridRow="1 / 3"
+        fontSize={pitWidth / 6}
       />
       <StoreView
         context={context}
         pitViewModel={revert ? player1BankViewModel : player2BankViewModel}
         gridColumn="1 / 2"
         gridRow="1 / 3"
+        fontSize={pitWidth / 6}
       />
       {revert ? player1Pits?.reverse() : player2Pits?.reverse()}
       {revert ? player2Pits : player1Pits}

apps/frontend/src/components/board/PitView.tsx

@@ -1,20 +1,56 @@
 import * as React from "react";
 import { FunctionComponent } from "react";
+import { getColorByBrightness, PitViewModel } from "@mancala/core";
+import { Context } from "../../context/context";
 import Util from "../../util/Util";
-import { PitViewModel } from "@mancala/core";
 import StoneView from "./StoneView";
 
-const PitView: FunctionComponent<{
+interface PitViewProps {
   pitViewModel: PitViewModel;
+  context: Context;
   onClick: () => void;
-}> = ({ pitViewModel, onClick }) => {
+  fontSize: number;
+}
+
+const PitView: FunctionComponent<PitViewProps> = ({ pitViewModel, context, onClick, fontSize }) => {
+  const textColor = getColorByBrightness(
+    pitViewModel.pitColor,
+    context.themeManager.theme.textColor,
+    context.themeManager.theme.textLightColor
+  );
+
   const stones = [...Util.range(pitViewModel.stoneCount)].map((i, index) => (
     <StoneView key={index} color={pitViewModel.stoneColor} />
   ));
 
   return (
-        <div className="pit" onClick={onClick} style={{ background: pitViewModel.pitColor }}>
+    <div
+      className="pit"
+      onClick={onClick}
+      style={{
+        background: pitViewModel.pitColor,
+        position: "relative",
+      }}
+    >
       {stones}
+      {pitViewModel.stoneCount > 0 && (
+        <span
+          className="stone-count"
+          style={{
+            color: textColor,
+            position: "absolute",
+            bottom: fontSize / 2,
+            left: "50%",
+            transform: "translateX(-50%)",
+            fontFamily: "monospace",
+            fontWeight: "bold",
+            fontSize: fontSize,
+            pointerEvents: "none",
+          }}
+        >
+          {pitViewModel.stoneCount}
+        </span>
+      )}
       <style jsx>{`
         .pit {
           margin: 5px;
@@ -27,6 +63,17 @@ const PitView: FunctionComponent<{
           justify-content: center;
           justify-items: center;
           flex-wrap: wrap;
+          position: relative;
+        }
+        .stone-count {
+          position: absolute;
+          bottom: 1.0vw;
+          left: 50%;
+          transform: translateX(-50%);
+          font-family: monospace;
+          font-size: 1.5vw;
+          font-weight: bold;
+          pointer-events: none;
         }
       `}</style>
     </div>

apps/frontend/yarn.lock

@@ -2019,10 +2019,10 @@ uuid@^10.0.0:
   resolved "https://registry.yarnpkg.com/uuid/-/uuid-10.0.0.tgz#5a95aa454e6e002725c79055fd42aaba30ca6294"
   integrity sha512-8XkAphELsDnEGrDxUOHB3RGvXz6TeuYSGEZBOjtTtPm2lwhGBjLgOzLHB63IUWfBpNucQjND6d3AOudO+H3RWQ==
 
-uuid@^8.3.2:
+uuid@^11.0.3:
-  version "8.3.2"
+  version "11.1.1"
-  resolved "https://registry.yarnpkg.com/uuid/-/uuid-8.3.2.tgz#80d5b5ced271bb9af6c445f21a1a04c606cefbe2"
+  resolved "https://registry.yarnpkg.com/uuid/-/uuid-11.1.1.tgz#f6d81d2e1c65d00762e5e29b16c5d2d995e208ad"
-  integrity sha512-+NYs2QeMWy+GWFOEm9xnn6HCDp0l7QBD7ml8zLUmJ+93Q5NF0NocErnwkTkXVFNiX3/fpC6afS8Dhb/gz7R7eg==
+  integrity sha512-vIYxrBCC/N/K+Js3qSN88go7kIfNPssr/hHCesKCQNAjmgvYS2oqr69kIufEG+O4+PfezOH4EbIeHCfFov8ZgQ==
 
 v8-compile-cache@^2.0.0:
   version "2.3.0"

apps/mobile/package.json

@@ -21,8 +21,8 @@
     "react-i18next": "^14.1.0",
     "react-native": "0.82.1",
     "react-native-get-random-values": "^1.11.0",
-    "react-native-mmkv": "^4.0.0",
+    "react-native-mmkv": "^4.3.2",
-    "react-native-nitro-modules": "^0.31.6",
+    "react-native-nitro-modules": "^0.36.1",
     "react-native-safe-area-context": "^5.5.2",
     "react-native-screens": "^4.18.0",
     "react-native-snackbar": "^2.6.2",

apps/mobile/src/components/board/BoardView.tsx

@@ -17,9 +17,9 @@ const BoardView: FunctionComponent<{
   boardViewModel: BoardViewModel;
   revert: boolean;
   onPitSelect: (index: number, pit: Pit) => void;
-}> = ({ game, context, boardId, boardViewModel, revert, onPitSelect: onPitSelect }) => {
+}> = ({ game, context, boardId, boardViewModel, revert, onPitSelect }) => {
   const windowWidth = Dimensions.get('window').width;
-  const windowHeight = Dimensions.get('window').height;
+  const windowHeight = Dimensions.get('window').height; // Currently unused but kept for potential future use
 
   const boardMargin = 0;
   const boardPadding = 10;
@@ -35,14 +35,20 @@ const BoardView: FunctionComponent<{
     const stones = [...Util.range(pitViewModel.stoneCount)].map((i, index) => (
       <StoneView key={index} color={pitViewModel.stoneColor} style={{ width: pitWidth / 10, height: pitWidth / 10 }} />
     ));
-    return <PitView 
+    return (
+      <PitView
         key={key}
         pitViewModel={pitViewModel}
+        context={context}
         onClick={() => onPitSelect(pit.index, pit)}
-    style={{width: pitWidth, height: pitWidth}} > 
+        style={{ width: pitWidth, height: pitWidth }}
+        fontSize={pitWidth / 5} // keep original divisor as per revert request
+      >
         {stones}
-    </PitView>;
+      </PitView>
+    );
   };
+
   const createPitViewList = (pits: Pit[]) => pits.map((pit, index) => createPitView(index, pit, boardViewModel.pits[pit.index]));
 
   const player1Pits = createPitViewList(mancalaGame?.board.player1Pits);
@@ -54,22 +60,26 @@ const BoardView: FunctionComponent<{
 
   const primaryStoreViewModel = revert ? player1BankViewModel : player2BankViewModel;
   const secondaryStoreViewModel = revert ? player2BankViewModel : player1BankViewModel;
+
   const stonesPrimaryStore = [...Util.range(primaryStoreViewModel.stoneCount)].map((i, index) => (
     <StoneView key={index} color={primaryStoreViewModel.stoneColor} style={{ width: pitWidth / 10, height: pitWidth / 10 }} />
   ));
+
   const stonesSecondaryStore = [...Util.range(secondaryStoreViewModel.stoneCount)].map((i, index) => (
     <StoneView key={index} color={secondaryStoreViewModel.stoneColor} style={{ width: pitWidth / 10, height: pitWidth / 10 }} />
   ));
 
   return (
-    <View style={{
+    <View
+      style={{
         backgroundColor: theme.boardColor,
         padding: boardPadding,
         borderRadius: 20,
         flexDirection: "row",
         gap: gap,
         height: "auto",
-    }}>
+      }}
+    >
       <StoreView
         context={context}
         pitViewModel={primaryStoreViewModel}
@@ -79,20 +89,24 @@ const BoardView: FunctionComponent<{
         {stonesPrimaryStore}
       </StoreView>
       <View style={{ flex: 6 }}>
-        <View style={{
+        <View
+          style={{
             flexDirection: "row",
-          width: '100%',
+            width: "100%",
             flex: 1,
-          gap: gap
+            gap: gap,
-        }}>
+          }}
+        >
           {revert ? player1Pits?.reverse() : player2Pits?.reverse()}
         </View>
-        <View style={{
+        <View
+          style={{
             flexDirection: "row",
-          width: '100%',
+            width: "100%",
             flex: 1,
-          gap: gap
+            gap: gap,
-        }}>
+          }}
+        >
           {revert ? player2Pits : player1Pits}
         </View>
       </View>

apps/mobile/src/components/board/PitView.tsx

@@ -1,35 +1,51 @@
 import * as React from "react";
 import { FunctionComponent } from "react";
-import { PitViewModel } from "@mancala/core";
+import { PitViewModel, getColorByBrightness } from "@mancala/core";
-import { Pressable, View, ViewStyle } from "react-native";
+import { Context } from "../../context/context";
+import { Pressable, View, ViewStyle, Text } from "react-native";
+import Util from "../../util/Util";
+import StoneView from "./StoneView";
 
-const PitView: FunctionComponent<{
+interface PitViewProps {
   pitViewModel: PitViewModel;
+  context: Context;
   onClick: () => void;
-    style: ViewStyle,
+  style: ViewStyle;
-    children: React.ReactNode
+  fontSize: number;
-}> = ({ pitViewModel, onClick, style, children }) => {
+}
-    return (
-        <View style={[style, {
-            backgroundColor: pitViewModel.pitColor,
-            borderRadius: 100,
-            //justifyItems: 'center',
-        }]}>
-            <Pressable onPress={onClick} style={{
-                width: "100%",
-                height: "100%",
-                padding: 5,
-                display: 'flex',
-                alignItems: 'center',
-                alignContent: 'center',
-                justifyContent: 'center',
-                flexDirection: "row",
-                flexWrap: "wrap"
-            }}>
-                {children}
-            </Pressable>
-        </View>
 
+const PitView: FunctionComponent<PitViewProps> = ({ pitViewModel, context, onClick, style, fontSize }) => {
+  const textColor = getColorByBrightness(
+    pitViewModel.pitColor,
+    context.themeManager.theme.textColor,
+    context.themeManager.theme.textLightColor
+  );
+
+  const stones = [...Util.range(pitViewModel.stoneCount)].map((i, index) => (
+    <StoneView key={index} color={pitViewModel.stoneColor} style={{ width: style.width ? style.width / 10 : 10, height: style.width ? style.width / 10 : 10 }} />
+  ));
+
+  return (
+    <View style={[style, { backgroundColor: pitViewModel.pitColor, borderRadius: 100, position: "relative" }] }>
+      <Pressable onPress={onClick} style={{ width: "100%", height: "100%", padding: 5, display: "flex", alignItems: "center", alignContent: "center", justifyContent: "center", flexDirection: "row", flexWrap: "wrap" }}>
+        {stones}
+      </Pressable>
+      {pitViewModel.stoneCount > 0 && (
+        <Text
+          style={{
+            color: textColor,
+            position: "absolute",
+            bottom: fontSize / 2,
+            alignSelf: "center",
+            fontFamily: "monospace",
+            fontWeight: "bold",
+            fontSize: fontSize,
+          }}
+        >
+          {pitViewModel.stoneCount}
+        </Text>
+      )}
+    </View>
   );
 };