Add missing required tandems, see #216

phetsims · Jan 23, 2020 · 46a0c77 · 46a0c77
1 parent fb49c9d
commit 46a0c77
Show file tree

Hide file tree

Showing 8 changed files with 564 additions and 14 deletions.
diff --git a/js/common/view/EFACTemperatureAndColorSensorNode.js b/js/common/view/EFACTemperatureAndColorSensorNode.js
@@ -37,7 +37,7 @@ define( require => {
         cursor: 'pointer',
 
         // phet-io
-        tandem: Tandem.OPTIONAL
+        tandem: Tandem.REQUIRED
       }, options );
 
       super( options );
@@ -71,7 +71,8 @@ define( require => {
           },
           endDrag: () => {
             temperatureAndColorSensor.userControlledProperty.set( false );
-          }
+          },
+          tandem: options.tandem.createTandem( 'inputListener' )
         } ) );
       }
     }

diff --git a/js/intro/view/BeakerContainerView.js b/js/intro/view/BeakerContainerView.js
@@ -94,7 +94,8 @@ define( require => {
         this.grabNode,
         modelViewTransform,
         constrainPosition,
-        model.isPlayingProperty
+        model.isPlayingProperty,
+        options.tandem.createTandem( 'inputListener' )
       ) );
     }
 

diff --git a/js/intro/view/BlockNode.js b/js/intro/view/BlockNode.js
@@ -278,7 +278,8 @@ define( require => {
         this,
         modelViewTransform,
         constrainPosition,
-        simIsPlayingProperty
+        simIsPlayingProperty,
+        options.tandem.createTandem( 'inputListener' )
       ) );
     }
 

diff --git a/js/intro/view/ThermalElementDragHandler.js b/js/intro/view/ThermalElementDragHandler.js
@@ -19,13 +19,15 @@ define( require => {
      * @param {ModelViewTransform2} modelViewTransform
      * @param {function} constrainPosition
      * @param {BooleanProperty} simIsPlayingProperty
+     * @param {Tandem} tandem
      */
     constructor(
       modelElement,
       screenViewChildNode,
       modelViewTransform,
       constrainPosition,
-      simIsPlayingProperty
+      simIsPlayingProperty,
+      tandem
     ) {
 
       const dragStartOffset = new Vector2( 0, 0 );
@@ -35,7 +37,6 @@ define( require => {
 
         // allow moving a finger (touch) across a screenViewChildNode to pick it up
         allowTouchSnag: true,
-
         start: event => {
 
           // make sure the sim is playing when an element is grabbed - this will resume the sim if it is paused
@@ -49,18 +50,17 @@ define( require => {
             dragStartPosition.y - modelElementViewPosition.y
           );
         },
-
         drag: event => {
           const dragPosition = currentTarget.globalToParentPoint( event.pointer.point );
           const modelElementViewPosition = dragPosition.minus( dragStartOffset );
           const modelElementPosition = modelViewTransform.viewToModelPosition( modelElementViewPosition );
           modelElement.positionProperty.set( constrainPosition( modelElement, modelElementPosition ) );
         },
-
         end: () => {
           modelElement.userControlledProperty.set( false );
           currentTarget = null;
-        }
+        },
+        tandem: tandem
       } );
     }
   }