@cornerstonejs/tools 1.43.5 → 1.43.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cornerstonejs/tools",
-  "version": "1.43.5",
+  "version": "1.43.6",
   "description": "Cornerstone3D Tools",
   "main": "src/index.ts",
   "types": "dist/types/index.d.ts",
@@ -29,7 +29,7 @@
     "webpack:watch": "webpack --mode development --progress --watch  --config ./.webpack/webpack.dev.js"
   },
   "dependencies": {
-    "@cornerstonejs/core": "^1.43.5",
+    "@cornerstonejs/core": "^1.43.6",
     "comlink": "^4.4.1",
     "lodash.clonedeep": "4.5.0",
     "lodash.get": "^4.4.2"
@@ -53,5 +53,5 @@
     "type": "individual",
     "url": "https://ohif.org/donate"
   },
-  "gitHead": "b63236689d4dc65ccfa11ba34a516991d17e726c"
+  "gitHead": "66da8823e7582d11c671c88902b2b2be8d6f9617"
 }

package/src/stateManagement/annotation/annotationState.ts

@@ -80,18 +80,18 @@ function addAnnotation(
   }
 
   const manager = getAnnotationManager();
-  const groupKey = manager.getGroupKey(annotationGroupSelector);
-
-  manager.addAnnotation(annotation, groupKey);
 
   // if the annotation manager selector is an element, trigger the
   // annotation added event for that element.
   if (annotationGroupSelector instanceof HTMLDivElement) {
+    const groupKey = manager.getGroupKey(annotationGroupSelector);
+    manager.addAnnotation(annotation, groupKey);
     triggerAnnotationAddedForElement(annotation, annotationGroupSelector);
   } else {
     // if no element is provided, render all viewports that have the
     // same frame of reference.
     // Todo: we should do something else here for other types of annotation managers.
+    manager.addAnnotation(annotation);
     triggerAnnotationAddedForFOR(annotation);
   }
 

package/src/stateManagement/annotation/helpers/state.ts

@@ -41,14 +41,12 @@ function triggerAnnotationAddedForFOR(annotation: Annotation) {
   const { toolName } = annotation.metadata;
 
   const toolGroups = getToolGroupsWithToolName(toolName);
-
   if (!toolGroups.length) {
     return;
   }
 
   // Find the viewports in the toolGroups who has the same FrameOfReferenceUID
   const viewportsToRender = [];
-
   toolGroups.forEach((toolGroup) => {
     toolGroup.viewportsInfo.forEach((viewportInfo) => {
       const { renderingEngineId, viewportId } = viewportInfo;
@@ -63,19 +61,17 @@ function triggerAnnotationAddedForFOR(annotation: Annotation) {
     });
   });
 
+  const eventType = Events.ANNOTATION_ADDED;
+  const eventDetail: AnnotationAddedEventDetail = { annotation };
+
   if (!viewportsToRender.length) {
+    triggerEvent(eventTarget, eventType, eventDetail);
     return;
   }
 
-  const eventType = Events.ANNOTATION_ADDED;
-
   viewportsToRender.forEach(({ renderingEngineId, viewportId }) => {
-    const eventDetail: AnnotationAddedEventDetail = {
-      annotation,
-      viewportId,
-      renderingEngineId,
-    };
-
+    eventDetail.viewportId = viewportId;
+    eventDetail.renderingEngineId = renderingEngineId;
     triggerEvent(eventTarget, eventType, eventDetail);
   });
 }

package/src/types/EventTypes.ts

@@ -104,9 +104,9 @@ type ToolActivatedEventDetail = {
  */
 type AnnotationAddedEventDetail = {
   /** unique id of the viewport */
-  viewportId: string;
+  viewportId?: string;
   /** unique id of the rendering engine */
-  renderingEngineId: string;
+  renderingEngineId?: string;
   /** The annotation that is being added to the annotations manager. */
   annotation: Annotation;
 };