Skip to content

Commit

Permalink
Documentation and add REVIEW comment, see #299
Browse files Browse the repository at this point in the history
  • Loading branch information
samreid committed Jun 30, 2020
1 parent c33bd5a commit 20e7236
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions js/atomic-interactions/view/AtomicInteractionsControlPanel.js
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,8 @@ function AtomicInteractionsControlPanel( dualAtomModel, enableHeterogeneousAtoms
],
spacing: 5
} );

// REVIEW: Variable name doesn't match tandem name
titleText = [ pinnedNodeText, new Text( movingString, {
font: new PhetFont( 10 ),
maxWidth: maxWidthOfTitleText,
Expand All @@ -183,6 +185,7 @@ function AtomicInteractionsControlPanel( dualAtomModel, enableHeterogeneousAtoms
} );
};

// REVIEW: variable name doesn't match tandem name
const aquaRadioButtonsGroup = new AquaRadioButtonGroup(
dualAtomModel.atomPairProperty,
[
Expand Down
4 changes: 2 additions & 2 deletions js/common/model/MoleculeForceAndMotionDataSetIO.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
// Copyright 2020, University of Colorado Boulder

/**
* IO type for MoleculeForceAndMotionDataSet, uses "data type" serialization where `fromStateObject will return of
* new instance.
* IO type for MoleculeForceAndMotionDataSet, uses "data type" serialization where `fromStateObject returns a new
* instance.
*
* @author John Blanco (PhET Interactive Simulations)
* @author Michael Kauzmann (PhET Interactive Simulations)
Expand Down

0 comments on commit 20e7236

Please sign in to comment.