chore: remove touch integration

Closes #796

assets/diagram-js.css

@@ -749,22 +749,13 @@ marker.djs-dragger tspan {
 }
 
 /**
- * touch
+ * bendpoints
  */
-
-.djs-shape,
-.djs-connection {
-  touch-action: none;
-}
-
 .djs-segment-dragger,
 .djs-bendpoint {
   display: none;
 }
 
-/**
- * bendpoints
- */
 .djs-segment-dragger .djs-visual {
   display: none;
 

lib/features/connect/Connect.js

@@ -122,7 +122,7 @@ export default function Connect(eventBus, dragging, modeling, rules) {
   /**
    * Start connect operation.
    *
-   * @param {MouseEvent|TouchEvent} event
+   * @param {MouseEvent} event
    * @param {Element} start
    * @param {Point} [connectionStart]
    * @param {boolean} [autoActivate=false]

lib/features/dragging/Dragging.js

@@ -43,13 +43,6 @@ function preventDefault(event) {
   event.preventDefault();
 }
 
-function isTouchEvent(event) {
-
-  // check for TouchEvent being available first
-  // (i.e. not available on desktop Firefox)
-  return typeof TouchEvent !== 'undefined' && event instanceof TouchEvent;
-}
-
 function getLength(point) {
   return Math.sqrt(Math.pow(point.x, 2) + Math.pow(point.y, 2));
 }
@@ -174,8 +167,7 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
       assign(
         {},
         dragContext.payload,
-        dragContext.data,
-        { isTouch: dragContext.isTouch }
+        dragContext.data
       )
     );
 
@@ -332,10 +324,6 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
     end(event);
   }
 
-  function trapTouch(event) {
-    move(event);
-  }
-
   // update the drag events model element (`hover`) and graphical element (`hoverGfx`)
   // properties during hover and out and fire {prefix}.hover and {prefix}.out properties
   // respectively
@@ -410,11 +398,6 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
 
     domEvent.unbind(document, 'keyup', checkCancel);
 
-    domEvent.unbind(document, 'touchstart', trapTouch, true);
-    domEvent.unbind(document, 'touchcancel', cancel, true);
-    domEvent.unbind(document, 'touchmove', move, true);
-    domEvent.unbind(document, 'touchend', end, true);
-
     eventBus.off('element.hover', hover);
     eventBus.off('element.out', out);
 
@@ -441,7 +424,7 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
    * If `localPosition` is given, drag events will be emitted
    * relative to it.
    *
-   * @param {MouseEvent|TouchEvent} [event]
+   * @param {MouseEvent} [event]
    * @param {Point} [relativeTo] actual diagram local position this drag operation should start at
    * @param {string} prefix
    * @param {Object} [options]
@@ -465,8 +448,7 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
         originalEvent,
         globalStart,
         localStart,
-        endDrag,
-        isTouch;
+        endDrag;
 
     if (options.trapClick) {
       endDrag = trapClickAndEnd;
@@ -495,41 +477,28 @@ export default function Dragging(eventBus, canvas, selection, elementRegistry) {
       relativeTo = localStart;
     }
 
-    isTouch = isTouchEvent(originalEvent);
-
     context = assign({
       prefix: prefix,
       data: data,
       payload: {},
       globalStart: globalStart,
       displacement: deltaPos(relativeTo, localStart),
-      localStart: localStart,
-      isTouch: isTouch
+      localStart: localStart
     }, options);
 
     // skip dom registration if trigger
     // is set to manual (during testing)
     if (!options.manual) {
 
       // add dom listeners
+      domEvent.bind(document, 'mousemove', move);
 
-      if (isTouch) {
-        domEvent.bind(document, 'touchstart', trapTouch, true);
-        domEvent.bind(document, 'touchcancel', cancel, true);
-        domEvent.bind(document, 'touchmove', move, true);
-        domEvent.bind(document, 'touchend', end, true);
-      } else {
-
-        // assume we use the mouse to interact per default
-        domEvent.bind(document, 'mousemove', move);
+      // prevent default browser drag and text selection behavior
+      domEvent.bind(document, 'dragstart', preventDefault);
+      domEvent.bind(document, 'selectstart', preventDefault);
 
-        // prevent default browser drag and text selection behavior
-        domEvent.bind(document, 'dragstart', preventDefault);
-        domEvent.bind(document, 'selectstart', preventDefault);
-
-        domEvent.bind(document, 'mousedown', endDrag, true);
-        domEvent.bind(document, 'mouseup', endDrag, true);
-      }
+      domEvent.bind(document, 'mousedown', endDrag, true);
+      domEvent.bind(document, 'mouseup', endDrag, true);
 
       domEvent.bind(document, 'keyup', checkCancel);
 

lib/features/interaction-events/InteractionEvents.js

@@ -76,7 +76,7 @@ export default function InteractionEvents(eventBus, elementRegistry, styles) {
    * Fire an interaction event.
    *
    * @param {string} type local event name, e.g. element.click.
-   * @param {MouseEvent|TouchEvent} event native event
+   * @param {MouseEvent} event native event
    * @param {Element} [element] the diagram element to emit the event on;
    *                                   defaults to the event target
    */
@@ -159,7 +159,7 @@ export default function InteractionEvents(eventBus, elementRegistry, styles) {
    * on the target shape or connection.
    *
    * @param {string} eventName the name of the triggered DOM event
-   * @param {MouseEvent|TouchEvent} event
+   * @param {MouseEvent} event
    * @param {Element} targetElement
    */
   function triggerMouseEvent(eventName, event, targetElement) {
@@ -508,8 +508,7 @@ InteractionEvents.$inject = [
  */
 
 /**
- * An event indicating that the context menu action is triggered
- * via mouse or touch controls.
+ * An event indicating that the context menu action is triggered via mouse.
  *
  * @event element.contextmenu
  *

lib/features/move/Move.js

@@ -201,7 +201,7 @@ export default function MoveEvents(
   /**
    * Start move.
    *
-   * @param {MouseEvent|TouchEvent} event
+   * @param {MouseEvent} event
    * @param {Shape} element
    * @param {boolean} [activate]
    * @param {Object} [context]

lib/features/resize/Resize.js

@@ -192,7 +192,7 @@ Resize.prototype.canResize = function(context) {
  * You may specify additional contextual information and must specify a
  * resize direction during activation of the resize event.
  *
- * @param {MouseEvent|TouchEvent} event
+ * @param {MouseEvent} event
  * @param {Shape} shape
  * @param {Object|Direction} contextOrDirection
  */

lib/features/resize/ResizeHandles.js

@@ -96,7 +96,6 @@ ResizeHandles.prototype.makeDraggable = function(element, gfx, direction) {
   }
 
   domEvent.bind(gfx, 'mousedown', startResize);
-  domEvent.bind(gfx, 'touchstart', startResize);
 };
 
 

lib/features/space-tool/SpaceTool.js

@@ -167,7 +167,7 @@ SpaceTool.$inject = [
 /**
  * Activate space tool selection.
  *
- * @param {MouseEvent|TouchEvent} event
+ * @param {MouseEvent} event
  * @param {boolean} autoActivate
  * @param {boolean} reactivate
  */
@@ -187,7 +187,7 @@ SpaceTool.prototype.activateSelection = function(event, autoActivate, reactivate
 /**
  * Activate space tool make space.
  *
- * @param {MouseEvent|TouchEvent} event
+ * @param {MouseEvent} event
  */
 SpaceTool.prototype.activateMakeSpace = function(event) {
   this._dragging.init(event, 'spaceTool', {
@@ -215,7 +215,7 @@ SpaceTool.prototype.makeSpace = function(movingShapes, resizingShapes, delta, di
 /**
  * Initialize make space and return true if that was successful.
  *
- * @param {MouseEvent|TouchEvent} event
+ * @param {MouseEvent} event
  * @param {Object} context
  *
  * @return {boolean}