Refactor addManipulationBindings function to accept options object

cornerstonejs · Feb 6, 2024 · 72bb717 · 72bb717
1 parent 5b16161
commit 72bb717
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/utils/demo/helpers/addManipulationBindings.ts b/utils/demo/helpers/addManipulationBindings.ts
@@ -27,23 +27,27 @@ let registered = false;
  */
 export default function addManipulationBindings(
   toolGroup,
-  register = true,
-  options?
+  options: {
+    enableShiftClickZoom?: boolean;
+    is3DViewport?: boolean;
+  }
 ) {
   const zoomBindings: Types.IToolBinding[] = [
     {
       mouseButton: MouseBindings.Secondary,
     },
   ];
 
-  if (options?.enableShiftClickZoom === true) {
+  const { is3DViewport = false, enableShiftClickZoom = false } = options;
+
+  if (enableShiftClickZoom === true) {
     zoomBindings.push({
       mouseButton: MouseBindings.Primary, // Shift Left Click
       modifierKey: KeyboardBindings.Shift,
     });
   }
 
-  if (register) {
+  if (!registered) {
     cornerstoneTools.addTool(LengthTool);
     cornerstoneTools.addTool(StackScrollMouseWheelTool);
     cornerstoneTools.addTool(PanTool);
@@ -71,10 +75,6 @@ export default function addManipulationBindings(
       {
         mouseButton: MouseBindings.Auxiliary,
       },
-      {
-        mouseButton: MouseBindings.Primary, // Ctrl Left drag
-        modifierKey: KeyboardBindings.Ctrl,
-      },
     ],
   });
   toolGroup.setToolActive(ZoomTool.toolName, {