TODO ANSWER: yes, they should be optional, phetsims/chipper#1360

phetsims · Jan 30, 2023 · c7b070a · c7b070a
1 parent b9a0611
commit c7b070a
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/js/GroundNode.ts b/js/GroundNode.ts
@@ -12,7 +12,6 @@ import { Color, TColor } from '../../scenery/js/imports.js';
 import GradientBackgroundNode from './GradientBackgroundNode.js';
 import sceneryPhet from './sceneryPhet.js';
 
-// TODO CM: should these be optional, or should we remove the defaults? https://github.com/phetsims/chipper/issues/1360
 type SelfOptions = {
   topColor?: TColor;
   bottomColor?: TColor;

diff --git a/js/SkyNode.ts b/js/SkyNode.ts
@@ -12,7 +12,6 @@ import { Color, TColor } from '../../scenery/js/imports.js';
 import GradientBackgroundNode from './GradientBackgroundNode.js';
 import sceneryPhet from './sceneryPhet.js';
 
-// TODO CM: should these be optional, or should we remove the defaults? https://github.com/phetsims/chipper/issues/1360
 type SelfOptions = {
   topColor?: TColor;
   bottomColor?: TColor;