- Enhanced camera feed state management with region handling and mode changes

src/features/cameras/components/CameraSettings/RegionSelector.tsx

@@ -1,45 +1,57 @@
-import ColourPicker from "./ColourPicker";
 import type { PaintedCell, Region } from "../../../../types/types";
-import type { RefObject } from "react";
+import ColourPicker from "./ColourPicker";
+import { useCameraFeedContext } from "../../../../app/context/CameraFeedContext";
 
 type RegionSelectorProps = {
   regions: Region[];
   selectedRegionIndex: number;
-  onSelectRegion: (index: number) => void;
-  onChangeRegionColour: (index: number, colour: string) => void;
   mode: string;
-  onSelectMode: (mode: string) => void;
-  paintedCells: RefObject<Map<string, PaintedCell>>;
-  onAddRegion: () => void;
-  OnRemoveRegion: () => void;
+  cameraFeedID: "A" | "B" | "C";
 };
 
-const RegionSelector = ({
-  regions,
-  selectedRegionIndex,
-  onSelectRegion,
-  onChangeRegionColour,
-  mode,
-  onSelectMode,
-  paintedCells,
-  onAddRegion,
-  OnRemoveRegion,
-}: RegionSelectorProps) => {
+const RegionSelector = ({ regions, selectedRegionIndex, mode, cameraFeedID }: RegionSelectorProps) => {
+  const { dispatch } = useCameraFeedContext();
   const handleChange = (e: { target: { value: string } }) => {
-    onSelectMode(e.target.value);
+    dispatch({ type: "CHANGE_MODE", payload: { cameraFeedID: cameraFeedID, mode: e.target.value } });
   };
 
-  const handleAddClick = () => {
-    onAddRegion();
+  const handleAddRegionClick = () => {
+    const regionName = `Region ${regions.length + 1}`;
+    dispatch({
+      type: "ADD_NEW_REGION",
+      payload: { cameraFeedID: cameraFeedID, regionName: regionName, brushColour: "#ffffff" },
+    });
   };
 
-  const handleResetClick = () => {
-    const map = paintedCells.current;
-    map.clear();
+  const handleResetRegion = () => {
+    dispatch({
+      type: "RESET_PAINTED_CELLS",
+      payload: { cameraFeedID: cameraFeedID, paintedCells: new Map<string, PaintedCell>() },
+    });
   };
 
   const handleRemoveClick = () => {
-    OnRemoveRegion();
+    dispatch({
+      type: "REMOVE_REGION",
+      payload: { cameraFeedID: cameraFeedID, regionName: regions[selectedRegionIndex].name },
+    });
+  };
+
+  const handleModeChange = (newMode: string) => {
+    dispatch({ type: "CHANGE_MODE", payload: { cameraFeedID: cameraFeedID, mode: newMode } });
+  };
+
+  const handleRegionSelect = (index: number) => {
+    dispatch({ type: "SET_SELECTED_REGION_INDEX", payload: index });
+  };
+
+  const handleRegionColourChange = (index: number, newColour: string) => {
+    const regionName = regions[index].name;
+
+    dispatch({
+      type: "SET_SELECTED_REGION_COLOUR",
+      payload: { cameraFeedID: cameraFeedID, regionName: regionName, newColour: newColour },
+    });
   };
 
   return (
@@ -84,7 +96,7 @@ const RegionSelector = ({
       <div className="p-2 border border-gray-600 rounded-lg flex flex-col">
         <h2 className="text-2xl mb-2">Region Select</h2>
         <>
-          {regions.map((region, idx) => {
+          {regions?.map((region, idx) => {
             const isSelected = selectedRegionIndex === idx;
             const inputId = `region-${idx}`;
             return (
@@ -102,20 +114,20 @@ const RegionSelector = ({
                     name="region"
                     className="sr-only"
                     onChange={() => {
-                      onSelectMode("painter");
-                      onSelectRegion(idx);
+                      handleModeChange("painter");
+                      handleRegionSelect(idx);
                     }}
                   />
                   <span className="text-xl">{region.name}</span>
                 </div>
-                <ColourPicker colour={region.brushColour} setColour={(c: string) => onChangeRegionColour(idx, c)} />
+                <ColourPicker colour={region.brushColour} setColour={(c: string) => handleRegionColourChange(idx, c)} />
                 <p className="text-slate-400">{region.brushColour}</p>
               </label>
             );
           })}
         </>
         <div className=" mx-auto flex flex-row gap-4  mt-4">
-          <button className="border border-blue-900 bg-blue-700 px-4 rounded-md" onClick={handleAddClick}>
+          <button className="border border-blue-900 bg-blue-700 px-4 rounded-md" onClick={handleAddRegionClick}>
             Add Region
           </button>
           <button className="border border-red-900 px-4 rounded-md" onClick={handleRemoveClick}>
@@ -128,10 +140,10 @@ const RegionSelector = ({
         <div className="flex flex-col">
           <h2 className="text-2xl mb-2">Actions</h2>
           <button
-            onClick={handleResetClick}
+            onClick={handleResetRegion}
             className="mt-2 px-4 py-2 border border-red-600 rounded-md text-white bg-red-600 w-full md:w-[40%] hover:bg-red-700 hover:cursor-pointer"
           >
-            Reset Regions
+            Reset Region
           </button>
         </div>
       </div>