fix(tools): prevent fill interference when placing annotations

- Suppress hover/context menu for existing annotations when actively
  placing a new polygon or rectangle
- Add cross-tool awareness between polygon and rectangle tools
- Check getActiveWidget() in widget behaviors to defer to focused widget
- Ensure grabFocus properly sets hasFocus to maintain activeState

Author: PaulHax

src/components/tools/polygon/PolygonTool.vue

@@ -92,6 +92,7 @@ import { Tools } from '@/src/store/tools/types';
 import { getLPSAxisFromDir } from '@/src/utils/lps';
 import { LPSAxisDir } from '@/src/types/lps';
 import { usePolygonStore } from '@/src/store/tools/polygons';
+import { useRectangleStore } from '@/src/store/tools/rectangles';
 import {
   useContextMenu,
   useCurrentTools,
@@ -235,7 +236,31 @@ export default defineComponent({
 
     // ---  //
 
-    const { contextMenu, openContextMenu } = useContextMenu();
+    const { contextMenu, openContextMenu: baseOpenContextMenu } =
+      useContextMenu();
+
+    // Check if any rectangle is actively being placed
+    const rectangleStore = useRectangleStore();
+    const isAnyRectanglePlacing = () => {
+      return rectangleStore.tools.some(
+        (tool) => tool.placing && tool.firstPoint && tool.secondPoint
+      );
+    };
+
+    // Suppress context menu for non-placing polygons when actively placing
+    // or when a rectangle is actively being placed
+    const openContextMenu = (id: ToolID, event: any) => {
+      if (isAnyRectanglePlacing()) {
+        return;
+      }
+      if (placingTool.id.value && id !== placingTool.id.value) {
+        const placingToolData = activeToolStore.toolByID[placingTool.id.value];
+        if (placingToolData?.points?.length > 0) {
+          return;
+        }
+      }
+      baseOpenContextMenu(id, event);
+    };
 
     const currentTools = useCurrentTools(
       activeToolStore,
@@ -247,7 +272,24 @@ export default defineComponent({
       })
     );
 
-    const { onHover, overlayInfo } = useHover(currentTools, slice);
+    const { onHover: baseOnHover, overlayInfo } = useHover(currentTools, slice);
+
+    // Suppress hover for non-placing polygons when actively placing (has points)
+    // or when a rectangle is actively being placed
+    const onHover = (id: ToolID, event: any) => {
+      if (isAnyRectanglePlacing()) {
+        baseOnHover(id, { ...event, hovering: false });
+        return;
+      }
+      if (placingTool.id.value && id !== placingTool.id.value) {
+        const placingToolData = activeToolStore.toolByID[placingTool.id.value];
+        if (placingToolData?.points?.length > 0) {
+          baseOnHover(id, { ...event, hovering: false });
+          return;
+        }
+      }
+      baseOnHover(id, event);
+    };
 
     const mergePossible = computed(
       () => activeToolStore.mergeableTools.length >= 1

src/components/tools/rectangle/RectangleTool.vue

@@ -28,6 +28,7 @@ import { Tools } from '@/src/store/tools/types';
 import { getLPSAxisFromDir } from '@/src/utils/lps';
 import { LPSAxisDir } from '@/src/types/lps';
 import { useRectangleStore } from '@/src/store/tools/rectangles';
+import { usePolygonStore } from '@/src/store/tools/polygons';
 import {
   useCurrentTools,
   useContextMenu,
@@ -39,6 +40,7 @@ import AnnotationInfo from '@/src/components/tools/AnnotationInfo.vue';
 import { useFrameOfReference } from '@/src/composables/useFrameOfReference';
 import { Maybe } from '@/src/types';
 import { useSliceInfo } from '@/src/composables/useSliceInfo';
+import { ToolID } from '@/src/types/annotation-tool';
 import { watchImmediate } from '@vueuse/core';
 import RectangleWidget2D from './RectangleWidget2D.vue';
 
@@ -121,7 +123,8 @@ export default defineComponent({
 
     // --- //
 
-    const { contextMenu, openContextMenu } = useContextMenu();
+    const { contextMenu, openContextMenu: baseOpenContextMenu } =
+      useContextMenu();
 
     const currentTools = useCurrentTools(
       activeToolStore,
@@ -133,7 +136,31 @@ export default defineComponent({
       })
     );
 
-    const { onHover, overlayInfo } = useHover(currentTools, slice);
+    const { onHover: baseOnHover, overlayInfo } = useHover(currentTools, slice);
+
+    // Check if any polygon is actively being placed (has points)
+    const polygonStore = usePolygonStore();
+    const isAnyPolygonPlacing = () => {
+      return polygonStore.tools.some(
+        (tool) => tool.placing && tool.points.length > 0
+      );
+    };
+
+    // Suppress hover/context menu when a polygon is actively being placed
+    const onHover = (id: ToolID, event: any) => {
+      if (isAnyPolygonPlacing()) {
+        baseOnHover(id, { ...event, hovering: false });
+        return;
+      }
+      baseOnHover(id, event);
+    };
+
+    const openContextMenu = (id: ToolID, event: any) => {
+      if (isAnyPolygonPlacing()) {
+        return;
+      }
+      baseOpenContextMenu(id, event);
+    };
 
     return {
       tools: currentTools,

src/vtk/PolygonWidget/behavior.ts

@@ -192,8 +192,6 @@ export default function widgetBehavior(publicAPI: any, model: any) {
     if (model.widgetState.getPlacing() && manipulator) {
       // Dropping first point?
       if (model.widgetState.getHandles().length === 0) {
-        // update variables used by updateActiveStateHandle
-        model.activeState = model.widgetState.getMoveHandle();
         model._widgetManager.grabFocus(publicAPI);
       }
       updateActiveStateHandle(event);
@@ -252,6 +250,16 @@ export default function widgetBehavior(publicAPI: any, model: any) {
       model._widgetManager.disablePicking();
     }
 
+    // Don't emit hover events if another widget has focus (e.g., is placing)
+    const activeWidget = model._widgetManager.getActiveWidget();
+    if (activeWidget && activeWidget !== publicAPI) {
+      publicAPI.invokeHoverEvent({
+        ...event,
+        hovering: false,
+      });
+      return macro.VOID;
+    }
+
     publicAPI.invokeHoverEvent({
       ...event,
       hovering: !!model.activeState || checkOverFill(),
@@ -424,6 +432,12 @@ export default function widgetBehavior(publicAPI: any, model: any) {
       return macro.EVENT_ABORT;
     }
 
+    // If another widget has focus (e.g., is placing), don't show context menu
+    const activeWidget = model._widgetManager.getActiveWidget();
+    if (activeWidget && activeWidget !== publicAPI) {
+      return macro.VOID;
+    }
+
     const eventWithWidgetAction = {
       ...eventData,
       widgetActions: makeWidgetActions(eventData),

src/vtk/RulerWidget/behavior.ts

@@ -184,6 +184,16 @@ export default function widgetBehavior(publicAPI: any, model: any) {
       return macro.EVENT_ABORT;
     }
 
+    // Don't emit hover events if another widget has focus (e.g., is placing)
+    const activeWidget = model._widgetManager.getActiveWidget();
+    if (activeWidget && activeWidget !== publicAPI) {
+      publicAPI.invokeHoverEvent({
+        ...eventData,
+        hovering: false,
+      });
+      return macro.VOID;
+    }
+
     publicAPI.invokeHoverEvent({
       ...eventData,
       hovering: !!model.activeState || checkOverFill(),
@@ -220,6 +230,13 @@ export default function widgetBehavior(publicAPI: any, model: any) {
     ) {
       return macro.VOID;
     }
+
+    // If another widget has focus (e.g., is placing), don't show context menu
+    const activeWidget = model._widgetManager.getActiveWidget();
+    if (activeWidget && activeWidget !== publicAPI) {
+      return macro.VOID;
+    }
+
     publicAPI.invokeRightClickEvent(eventData);
     return macro.EVENT_ABORT;
   };