Fix moveNewHandle (#1209)

* Fix moveNewHandle * fix: 🐛 Fixes moveNewHandle for click-move-click interaction
cornerstonejs · Apr 2, 2020 · 1a627bf · 1a627bf
1 parent 24bca45
commit 1a627bf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 18 deletions.
diff --git a/src/manipulators/moveNewHandle.js b/src/manipulators/moveNewHandle.js
@@ -60,6 +60,8 @@ export default function(
     options
   );
 
+  options.hasMoved = false;
+
   const { element } = eventData;
 
   annotation.active = true;
@@ -135,6 +137,8 @@ function _moveHandler(
 ) {
   const { currentPoints, image, element, buttons } = evt.detail;
 
+  options.hasMoved = true;
+
   const page = currentPoints.page;
   const fingerOffset = -57;
   const targetLocation = external.cornerstone.pageToPixel(
@@ -218,6 +222,10 @@ function _moveEndHandler(
   const eventData = evt.detail;
   const { element, currentPoints } = eventData;
 
+  if (options.hasMoved === false) {
+    return;
+  }
+
   const page = currentPoints.page;
   const fingerOffset = -57;
   const targetLocation = external.cornerstone.pageToPixel(

diff --git a/src/store/modules/manipulatorStateModule.js b/src/store/modules/manipulatorStateModule.js
@@ -56,24 +56,6 @@ function removeEnabledElementCallback(element) {
   removeActiveManipulatorForElement(element);
 }
 
-//
-
-// const enabledElements = cornerstoneTools.store.state.enabledElements;
-
-// const cancelActiveManipulatorsForElement = cornerstoneTools.getModule(
-//   'manipulatorState'
-// ).setters.cancelActiveManipulatorsForElement;
-
-// enabledElements.forEach(element => {
-//   element.addEventListener('keydown', evt => {
-//     if (evt.keyCode === 27) {
-//       cancelActiveManipulatorsForElement(element);
-//     }
-//   });
-// });
-
-///
-
 function enabledElementCallback(element) {
   const { NEW_IMAGE } = external.cornerstone.EVENTS;