Added docs for FrictionModel.init, see #64

phetsims · Feb 14, 2018 · b04dc52 · b04dc52
1 parent 9bb9ba3
commit b04dc52
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/js/friction/model/FrictionModel.js b/js/friction/model/FrictionModel.js
@@ -225,6 +225,8 @@ define( function( require ) {
         self.tryToEvaporate();
       }
     } );
+
+    this.init();
   }
 
   friction.register( 'FrictionModel', FrictionModel );
@@ -270,8 +272,8 @@ define( function( require ) {
     },
 
     /**
-     * TODO: this must be called from the end of the view construction for unknown reasons, or atoms don't fly off
-     * TODO: Why?  it would be better if this could be called during the constructor.
+     * This must be called after MagnifierNode adds AtomNodes to the toEvaporateSample, or atoms don't fly off
+     * TODO: It would be better if this could be called during the constructor and didn't need a view step first
      * @public
      */
     init: function() {

diff --git a/js/friction/view/FrictionScreenView.js b/js/friction/view/FrictionScreenView.js
@@ -90,7 +90,8 @@ define( function( require ) {
       tandem: tandem.createTandem( 'resetAllButton' )
     } ) );
 
-    model.init(); // TODO: does this need to be called here?  If so, why?
+    // Initialize the model now that MagnifierNode has added the toEvaporateSample AtomNodes
+    model.init();
   }
 
   friction.register( 'FrictionScreenView', FrictionScreenView );