Added docs, see #86

phetsims · Nov 21, 2017 · 36dcc89 · 36dcc89
1 parent 8632522
commit 36dcc89
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/js/faradays-law/model/FaradaysLawModel.js b/js/faradays-law/model/FaradaysLawModel.js
@@ -125,7 +125,8 @@ define( function( require ) {
      */
     moveMagnetToPosition: function( position ) {
 
-      // TODO: why do we subtract 1 from each of these dimensions?
+      // Check the bounds of the magnet, but subtract 1 to prevent it from passing through the coils vertically
+      // see https://github.com/phetsims/faradays-law/issues/47
       var magnetBounds = new Bounds2(
         Math.min( position.x, this.magnet.positionProperty.get().x ),
         Math.min( position.y, this.magnet.positionProperty.get().y ),