experiment with allowTouchSnag:false for functions, #29

phetsims · May 18, 2016 · c531410 · c531410
1 parent 1a64a19
commit c531410
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/js/common/view/FunctionNode.js b/js/common/view/FunctionNode.js
@@ -30,7 +30,9 @@ define( function( require ) {
    */
   function FunctionNode( functionInstance, contentNode, container, builderNode, dragLayer, options ) {
 
-    options = options || {};
+    options = _.extend( {
+      allowTouchSnag: false // see https://github.com/phetsims/function-builder/issues/29
+    }, options );
 
     var thisNode = this;
 

diff --git a/js/common/view/MovableNode.js b/js/common/view/MovableNode.js
@@ -23,6 +23,8 @@ define( function( require ) {
   function MovableNode( movable, options ) {
 
     options = _.extend( {
+
+      allowTouchSnag: true, // {boolean} allow touch swipes across this Node to pick it up
       cursor: 'pointer',
       startDrag: null, // {function|null} Called at the start of each drag sequence
       endDrag: null, // {function|null} Called at the end of each drag sequence
@@ -53,8 +55,7 @@ define( function( require ) {
 
     this.addInputListener( new SimpleDragHandler( {
 
-      // allow touch swipes across this Node to pick it up
-      allowTouchSnag: true,
+      allowTouchSnag: options.allowTouchSnag,
 
       start: function( event, trail ) {