add canceling for road building (#284)

bcollazo · Sep 14, 2024 · 76442c7 · 76442c7
1 parent e4d8ca0
commit 76442c7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/ui/src/actions.js b/ui/src/actions.js
@@ -2,7 +2,7 @@ const ACTIONS = Object.freeze({
   SET_LEFT_DRAWER_OPENED: "SET_LEFT_DRAWER_OPENED",
   SET_GAME_STATE: "SET_GAME_STATE",
 
-  SET_IS_BUILDING_ROAD: "SET_IS_BUILDING_ROAD",
+  TOGGLE_BUILDING_ROAD: "TOGGLE_BUILDING_ROAD",
   SET_IS_BUILDING_SETTLEMENT: "SET_IS_BUILDING_SETTLEMENT",
   SET_IS_BUILDING_CITY: "SET_IS_BUILDING_CITY",
   SET_IS_PLAYING_MONOPOLY: "SET_IS_PLAYING_MONOPOLY",

diff --git a/ui/src/pages/ActionsToolbar.js b/ui/src/pages/ActionsToolbar.js
@@ -147,8 +147,8 @@ function PlayButtons() {
   const setIsBuildingCity = useCallback(() => {
     dispatch({ type: ACTIONS.SET_IS_BUILDING_CITY });
   }, [dispatch]);
-  const setIsBuildingRoad = useCallback(() => {
-    dispatch({ type: ACTIONS.SET_IS_BUILDING_ROAD });
+  const toggleBuildingRoad = useCallback(() => {
+    dispatch({ type: ACTIONS.TOGGLE_BUILDING_ROAD });
   }, [dispatch]);
   const buildItems = [
     {
@@ -169,7 +169,7 @@ function PlayButtons() {
     {
       label: "Road",
       disabled: !buildActionTypes.has("BUILD_ROAD"),
-      onClick: setIsBuildingRoad,
+      onClick: toggleBuildingRoad,
     },
   ];
 

diff --git a/ui/src/store.js b/ui/src/store.js
@@ -36,8 +36,8 @@ const StateProvider = ({ children }) => {
           isPlayingYearOfPlenty: false,
           isMovingRobber: false,
         };
-      case ACTIONS.SET_IS_BUILDING_ROAD:
-        return { ...state, isBuildingRoad: true };
+      case ACTIONS.TOGGLE_BUILDING_ROAD:
+        return { ...state, isBuildingRoad: !state.isBuildingRoad}
       case ACTIONS.SET_IS_BUILDING_SETTLEMENT:
         return { ...state, isBuildingSettlement: true };
       case ACTIONS.SET_IS_BUILDING_CITY: