Renamed var and revised docs, see #64

phetsims · Feb 13, 2018 · 288218c · 288218c
1 parent 9d6172f
commit 288218c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/js/friction/view/book/BookNode.js b/js/friction/view/book/BookNode.js
@@ -29,7 +29,7 @@ define( function( require ) {
   var bookTitleStringPattern = FrictionA11yStrings.bookTitleStringPattern.value;
 
   /**
-   * @param {FrictionModel} model TODO: after making TODO changes below, maybe the entire model doesn't need to be passed in
+   * @param {FrictionModel} model
    * @param {string} title - title that appears on the book spine
    * @param {Object} [options]
    * @constructor

diff --git a/js/friction/view/magnifier/AtomCanvasNode.js b/js/friction/view/magnifier/AtomCanvasNode.js
@@ -86,9 +86,8 @@ define( function( require ) {
     );
     context.fill();
 
-    // array that holds the AtomNode views
-    // TODO: visibility annotation
-    this.atoms = [];
+    // @private - array that holds the AtomNode views
+    this.atomCanvasNodeAtoms = [];
 
     // pre-allocated vector, improves performance
     // TODO: visibility annotation
@@ -110,8 +109,8 @@ define( function( require ) {
       var particleImageSize = FrictionConstants.ATOM_RADIUS * 2 * 1.1;
 
       // render each of the atoms on the canvas
-      for ( var i = 0; i < this.atoms.length; i++ ) {
-        var atom = this.atoms[ i ];
+      for ( var i = 0; i < this.atomCanvasNodeAtoms.length; i++ ) {
+        var atom = this.atomCanvasNodeAtoms[ i ];
         this.particleImagePosition.x = atom.currentX - particleImageSize / 2;
         this.particleImagePosition.y = atom.currentY - particleImageSize / 2;
         context.drawImage(
@@ -134,7 +133,7 @@ define( function( require ) {
      * @public
      */
     registerAtom: function( atom ) {
-      this.atoms.push( atom );
+      this.atomCanvasNodeAtoms.push( atom );
     }
   } );
 } );