change issue number associated with some TODO items #30

phetsims · Jul 19, 2017 · 0bbc5d6 · 0bbc5d6
1 parent 784f140
commit 0bbc5d6
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 7 deletions.
diff --git a/js/common/MPQueryParameters.js b/js/common/MPQueryParameters.js
@@ -29,7 +29,7 @@ define( function( require ) {
       defaultValue: 'RWB'
     },
 
-    //TODO delete when Real Molecules is fully implemented, see #15
+    //TODO delete this query parameter when Real Molecules is fully implemented, see #32
     // enables the 'Real Molecules' screen, for development
     realMolecules: { type: 'flag' }
   } );

diff --git a/js/common/view/MPOptionsNode.js b/js/common/view/MPOptionsNode.js
@@ -27,12 +27,13 @@ define( function( require ) {
     var surfaceColorControl = ( new SurfaceColorControl( MPConstants.GLOBAL_OPTIONS.surfaceColorProperty ) );
 
     var children = [
-      dipoleDirectionControl
+      dipoleDirectionControl,
+      surfaceColorControl
     ];
 
-    //TODO clean this up when Real Molecules screen is implemented, see #15
-    if ( MPQueryParameters.realMolecules ) {
-      children.push( surfaceColorControl );
+    //TODO remove the Surface Color option until Real Molecules screen is implemented, see #32
+    if ( !MPQueryParameters.realMolecules ) {
+      children.splice( children.indexOf( surfaceColorControl ), 1 );
     }
 
     VBox.call( this, {

diff --git a/js/realmolecules/view/RealMoleculesScreenView.js b/js/realmolecules/view/RealMoleculesScreenView.js
@@ -135,7 +135,7 @@ define( function( require ) {
       electronDensityColorKey.visible = ( surfaceType === SurfaceType.ELECTRON_DENSITY );
     } );
 
-    //TODO Hide everything and show a dialog until this screen is fully implemented, see #15
+    //TODO Hide everything and show a dialog until Real Molecules screen is fully implemented, see #32
     if ( MPQueryParameters.realMolecules ) {
       this.addChild( rootNode );
     }

diff --git a/js/realmolecules/view/UnderDevelopmentPlane.js b/js/realmolecules/view/UnderDevelopmentPlane.js
@@ -1,6 +1,6 @@
 // Copyright 2017, University of Colorado Boulder
 
-//TODO Delete when the 3D viewer is implemented, see #15
+//TODO Delete when the Real Molecules screen is completed, see #32
 /**
  * Used to disable the 'Real Molecules' screen and display a message indicating that it's under development.
  *