From c9a1c07c6fb01b7e234557cc6b2f921412596101 Mon Sep 17 00:00:00 2001 From: chrisklus Date: Mon, 5 Nov 2018 23:48:16 -0700 Subject: [PATCH] add review comments, see https://github.com/phetsims/graphing-quadratics/issues/43 --- js/explore/view/ExploreInteractiveEquationNode.js | 2 ++ .../view/FocusAndDirectrixInteractiveEquationNode.js | 2 ++ 2 files changed, 4 insertions(+) diff --git a/js/explore/view/ExploreInteractiveEquationNode.js b/js/explore/view/ExploreInteractiveEquationNode.js index 0972c5f1..16dfc05d 100644 --- 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 index 2ed0c312..161fde15 100644 --- 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' } );