From cfb714d5c1f31b23940a8f2e5e6b032521570106 Mon Sep 17 00:00:00 2001
From: jbphet <john.blanco@colorado.edu>
Date: Fri, 12 Jun 2020 14:33:13 -0600
Subject: [PATCH] fixed an issue where the lid could move by itself after and
 explosion followed by a return of the lid

---
 js/common/view/ParticleContainerNode.js | 13 ++++++++++---
 js/common/view/PointingHandNode.js      | 24 +++++++++++++++---------
 2 files changed, 25 insertions(+), 12 deletions(-)

diff --git a/js/common/view/ParticleContainerNode.js b/js/common/view/ParticleContainerNode.js
index 4c200daf..b1998672 100644
--- a/js/common/view/ParticleContainerNode.js
+++ b/js/common/view/ParticleContainerNode.js
@@ -174,15 +174,22 @@ function ParticleContainerNode( multipleParticleModel, modelViewTransform, optio
       end: () => {
 
         // Set the target size to the current size, which will stop any change in size that is currently underway.
-        multipleParticleModel.setTargetContainerHeight(
-          multipleParticleModel.containerHeightProperty.get()
-        );
+        if ( !multipleParticleModel.isExplodedProperty.value ) {
+          multipleParticleModel.setTargetContainerHeight( multipleParticleModel.containerHeightProperty.get() );
+        }
       },
 
       tandem: lidTandem.createTandem( 'lidDragListener' )
     } ) );
   }
 
+  // The particle container can explode while the lid is being dragged and, if that happens, cancel the interaction.
+  multipleParticleModel.isExplodedProperty.lazyLink( isExploded => {
+    if ( isExploded ) {
+      this.interruptSubtreeInput();
+    }
+  } );
+
   let pressureGaugeNode;
   if ( options.pressureGaugeEnabled ) {
 
diff --git a/js/common/view/PointingHandNode.js b/js/common/view/PointingHandNode.js
index eef3e51a..a1bd7d6e 100644
--- a/js/common/view/PointingHandNode.js
+++ b/js/common/view/PointingHandNode.js
@@ -23,13 +23,12 @@ import MultipleParticleModel from '../model/MultipleParticleModel.js';
 const WIDTH = 150; // in screen coords
 
 /**
- *
- * @param {MultipleParticleModel} multipleParticleModel - model of the simulation
+ * @param {PhaseChangesModel} phaseChangesModel - model of the simulation
  * @param {ModelViewTransform2} modelViewTransform to convert between model and view co-ordinate frames
  * @param {Object} [options]
  * @constructor
  */
-function PointingHandNode( multipleParticleModel, modelViewTransform, options ) {
+function PointingHandNode( phaseChangesModel, modelViewTransform, options ) {
 
   options = merge( {
     tandem: Tandem.REQUIRED
@@ -94,7 +93,7 @@ function PointingHandNode( multipleParticleModel, modelViewTransform, options )
     start: event => {
       startY = self.globalToParentPoint( event.pointer.point ).y;
       beingDragged = true;
-      containerSizeAtDragStart = multipleParticleModel.containerHeightProperty.get();
+      containerSizeAtDragStart = phaseChangesModel.containerHeightProperty.get();
       updateHintVisibility();
     },
 
@@ -102,7 +101,7 @@ function PointingHandNode( multipleParticleModel, modelViewTransform, options )
       endY = self.globalToParentPoint( event.pointer.point ).y;
 
       // Resize the container based on the amount that the node has moved.
-      multipleParticleModel.setTargetContainerHeight(
+      phaseChangesModel.setTargetContainerHeight(
         containerSizeAtDragStart + modelViewTransform.viewToModelDeltaY( endY - startY )
       );
       updateHintVisibility();
@@ -111,9 +110,9 @@ function PointingHandNode( multipleParticleModel, modelViewTransform, options )
     end: event => {
 
       // Set the target size to the current size, which will stop any change in size that is currently underway.
-      multipleParticleModel.setTargetContainerHeight(
-        multipleParticleModel.containerHeightProperty.get()
-      );
+      if ( !phaseChangesModel.isExplodedProperty.value ) {
+        phaseChangesModel.setTargetContainerHeight( phaseChangesModel.containerHeightProperty.get() );
+      }
       beingDragged = false;
       updateHintVisibility();
     },
@@ -121,6 +120,13 @@ function PointingHandNode( multipleParticleModel, modelViewTransform, options )
     tandem: options.tandem.createTandem( 'dragListener' )
   } ) );
 
+  // The particle container can explode while this is being dragged and, if that happens, cancel the interaction.
+  phaseChangesModel.isExplodedProperty.lazyLink( isExploded => {
+    if ( isExploded ) {
+      this.interruptSubtreeInput();
+    }
+  } );
+
   // add the listener that will show and hide the hint
   this.addInputListener( {
     enter: function() {
@@ -141,7 +147,7 @@ function PointingHandNode( multipleParticleModel, modelViewTransform, options )
   this.touchArea = this.localBounds.dilatedXY( 10, 10 );
 
   // Add a listener to update the individual arrow visibility.
-  multipleParticleModel.containerHeightProperty.link( function( particleContainerHeight ) {
+  phaseChangesModel.containerHeightProperty.link( function( particleContainerHeight ) {
     if ( particleContainerHeight === MultipleParticleModel.PARTICLE_CONTAINER_INITIAL_HEIGHT ) {
 
       // At the height limit, so only show the down arrow.