add review comments, see #43

phetsims · Nov 6, 2018 · c9a1c07 · c9a1c07
1 parent da5bebe
commit c9a1c07
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/js/explore/view/ExploreInteractiveEquationNode.js b/js/explore/view/ExploreInteractiveEquationNode.js
@@ -43,6 +43,8 @@ define( require => {
       // equation
       const equationNode = new EquationNode( aProperty, bProperty, cProperty, {
         tandem: options.tandem.createTandem( 'equationNode' ),
+        //REVIEW: suggested phetioDocumentation: 'the equation that changes as the sliders are adjusted'.
+        //see FocusAndDirectrixInteractiveEquationNode.js
         phetioDocumentation: 'the equation that changes as you adjust the sliders'
       } );
 

diff --git a/js/focusanddirectrix/view/FocusAndDirectrixInteractiveEquationNode.js b/js/focusanddirectrix/view/FocusAndDirectrixInteractiveEquationNode.js
@@ -43,6 +43,8 @@ define( require => {
       // equation
       const equationNode = new EquationNode( pProperty, hProperty, kProperty, {
         tandem: options.tandem.createTandem( 'equationNode' ),
+        //REVIEW: suggested phetioDocumentation: 'the equation that changes as the sliders are adjusted'. after a quick
+        //search, i'm not seeing any uses of second person for phet-io documentation. also see ExploreInteractiveEquationNode.js
         phetioDocumentation: 'the equation that changes as you adjust the sliders'
       } );