Update TODOs, see #6

phetsims · Feb 2, 2022 · c666db3 · c666db3
1 parent 9c749e9
commit c666db3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/js/common/CASScreen.ts b/js/common/CASScreen.ts
@@ -16,7 +16,6 @@ import CASModel from './model/CASModel.js';
 import CASScreenView from './view/CASScreenView.js';
 import { PhetioObjectOptions } from '../../../tandem/js/PhetioObject.js';
 
-// type CASScreenOptions = ScreenOptions; //TODO
 export type CASScreenOptions = PhetioObjectOptions & Required<Pick<PhetioObjectOptions, 'tandem'>>;
 
 // TODO: Can we delete this file or does it provide value???
@@ -25,7 +24,7 @@ class CASScreen extends Screen {
   constructor( createModel: () => CASModel, createView: ( m: CASModel ) => CASScreenView, providedOptions?: CASScreenOptions ) {
 
     const options = optionize<CASScreenOptions>( {
-      //TODO if you include homeScreenIcon or navigationBarIcon, use JOIST/ScreenIcon
+      // TODO if you include homeScreenIcon or navigationBarIcon, use JOIST/ScreenIcon
       backgroundColorProperty: CASColors.screenBackgroundColorProperty,
 
       // phet-io options

diff --git a/js/common/view/BackgroundNode.ts b/js/common/view/BackgroundNode.ts
@@ -14,7 +14,6 @@ import Property from '../../../../axon/js/Property.js';
 import Bounds2 from '../../../../dot/js/Bounds2.js';
 import CASColors from '../CASColors.js';
 
-// TODO: Do we still want a separate node for Sky and Ground?
 class BackgroundNode extends Rectangle {
   constructor( bottomY: number, visibleBoundsProperty: Property<Bounds2> ) {
     super( visibleBoundsProperty.value.centerX, visibleBoundsProperty.value.top, visibleBoundsProperty.value.centerX, bottomY );