Add right click to delete points in desktop mask/zone editor (#12744)

blakeblackshear · Aug 9, 2024 · 6620806 · 6620806
1 parent ecea5c2
commit 6620806
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/web/src/components/settings/PolygonCanvas.tsx b/web/src/components/settings/PolygonCanvas.tsx
@@ -114,6 +114,29 @@ export function PolygonCanvas({
     const mousePos = stage.getPointerPosition() ?? { x: 0, y: 0 };
     const intersection = stage.getIntersection(mousePos);
 
+    // right click on desktops to delete a point
+    if (
+      e.evt instanceof MouseEvent &&
+      e.evt.button === 2 &&
+      intersection?.getClassName() == "Circle"
+    ) {
+      const pointIndex = parseInt(intersection.name()?.split("-")[1]);
+      if (!isNaN(pointIndex)) {
+        const updatedPoints = activePolygon.points.filter(
+          (_, index) => index !== pointIndex,
+        );
+        updatedPolygons[activePolygonIndex] = {
+          ...activePolygon,
+          points: updatedPoints,
+          pointsOrder: activePolygon.pointsOrder?.filter(
+            (_, index) => index !== pointIndex,
+          ),
+        };
+        setPolygons(updatedPolygons);
+      }
+      return;
+    }
+
     if (
       activePolygon.points.length >= 3 &&
       intersection?.getClassName() == "Circle" &&
@@ -236,6 +259,9 @@ export function PolygonCanvas({
       onMouseDown={handleMouseDown}
       onTouchStart={handleMouseDown}
       onMouseOver={handleStageMouseOver}
+      onContextMenu={(e) => {
+        e.evt.preventDefault();
+      }}
     >
       <Layer>
         <Image

diff --git a/web/src/components/settings/PolygonEditControls.tsx b/web/src/components/settings/PolygonEditControls.tsx
@@ -80,7 +80,7 @@ export default function PolygonEditControls({
             <MdUndo className="text-secondary-foreground" />
           </Button>
         </TooltipTrigger>
-        <TooltipContent>Undo</TooltipContent>
+        <TooltipContent>Remove last point</TooltipContent>
       </Tooltip>
       <Tooltip>
         <TooltipTrigger asChild>