diff --git a/js/lab/view/GravityAccordionBox.js b/js/lab/view/GravityAccordionBox.js
index 4f2c22c..5b6ad6b 100644
--- a/js/lab/view/GravityAccordionBox.js
+++ b/js/lab/view/GravityAccordionBox.js
@@ -63,7 +63,7 @@ define( require => {
       gravitySlider.addMajorTick( MassesAndSpringsConstants.GRAVITY_RANGE.min, new Text( noneString, {
         font: MassesAndSpringsConstants.LABEL_FONT,
         tandem: tandem.createTandem( 'gravityNoneText' ), // REVIEW: Tandem should be for the 'Text', not a 'String'?
-                                                          // *REVIEW: review with JO and update updat
+                                                          // *REVIEW: review with JO and update
         maxWidth: MAX_WIDTH * 0.5
       } ) );
       gravitySlider.addMajorTick( MassesAndSpringsConstants.GRAVITY_RANGE.max, new Text( lotsString, {
diff --git a/js/lab/view/LabScreenView.js b/js/lab/view/LabScreenView.js
index 2494f32..a42a3f1 100644
--- a/js/lab/view/LabScreenView.js
+++ b/js/lab/view/LabScreenView.js
@@ -142,7 +142,7 @@ define( require => {
       ] );
       this.springSystemControlsNode.spacing = this.spacing * 1.2;
 
-      // @protected {PeriodTraceNode} REVIEW: If protected, is there a subtype of this referencing periodTraceNode?
+      // @private {PeriodTraceNode}
       this.periodTraceNode = new PeriodTraceNode( model.periodTrace, this.modelViewTransform, model.options.basicsVersion, {
         center: this.massEquilibriumLineNode.center
       } );
@@ -165,9 +165,9 @@ define( require => {
     }
 
     /**
-     * @param {number} dt
-     *
      * @public
+     *
+     * @param {number} dt
      */
     step( dt ) {
       this.periodTraceNode.step( dt, this.model.playingProperty );