Update comments, see #45

phetsims · Mar 4, 2022 · eb20d38 · eb20d38
1 parent ee1d082
commit eb20d38
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/js/common/CASConstants.ts b/js/common/CASConstants.ts
@@ -24,8 +24,9 @@ const CASConstants = {
   CHECKBOX_TEXT_MAX_WIDTH: 120,
 
   // TODO: Should this be declared in main and passed through?
-  // SR: I prefer that it is treated like a global, but
-
+  // SR: I prefer that it is treated like a global.  But in other sims I've advocated for passing it through.  But here,
+  // SR: I tried passing it through and it got ugly because it had to be passed to places that seem like they shouldn't need it
+  // SR: Maybe we should ask dev team?
   PLOT_TYPE_PROPERTY: new EnumerationProperty( CASQueryParameters.plotType === 'dotPlot' ? PlotType.DOT_PLOT : PlotType.LINE_PLOT, {
     tandem: Tandem.GLOBAL_VIEW.createTandem( 'plotTypeProperty' )
   } ),