From 2c0ada573c609d5a5e49596b32856be0ba1e82ad Mon Sep 17 00:00:00 2001 From: Sam Reid Date: Sat, 13 May 2023 09:25:59 -0600 Subject: [PATCH] Move TODOs, see https://github.com/phetsims/center-and-variability/issues/170 and https://github.com/phetsims/center-and-variability/issues/201 --- js/variability/view/VariabilityAccordionBox.ts | 2 +- js/variability/view/VariabilityPlotNode.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/js/variability/view/VariabilityAccordionBox.ts b/js/variability/view/VariabilityAccordionBox.ts index 7d513b58..8cbd2725 100644 --- a/js/variability/view/VariabilityAccordionBox.ts +++ b/js/variability/view/VariabilityAccordionBox.ts @@ -118,7 +118,7 @@ export default class VariabilityAccordionBox extends CAVAccordionBox { const madValueProperty = deriveValueProperty( vsm => vsm.madValueProperty ); const meanValueProperty = deriveValueProperty( vsm => vsm.meanValueProperty ); - // TODO: Why can't this infer the type parameter? See https://github.com/phetsims/center-and-variability/issues/170 + // TODO: Why can't this infer the type parameter? See https://github.com/phetsims/center-and-variability/issues/201 const readoutsToggleNode = new ToggleNode( model.selectedVariabilityProperty, [ { value: VariabilityMeasure.RANGE, tandemName: 'rangeReadoutToggleNode', diff --git a/js/variability/view/VariabilityPlotNode.ts b/js/variability/view/VariabilityPlotNode.ts index cf776232..0b108523 100644 --- a/js/variability/view/VariabilityPlotNode.ts +++ b/js/variability/view/VariabilityPlotNode.ts @@ -30,7 +30,7 @@ export default class VariabilityPlotNode extends Node { public constructor( model: VariabilityModel, sceneModel: VariabilitySceneModel, providedOptions: CAVPlotOptions ) { super( providedOptions ); - // TODO: Why can't it infer reasonable types here? See https://github.com/phetsims/center-and-variability/issues/170 + // TODO: Why can't it infer reasonable types here? See https://github.com/phetsims/center-and-variability/issues/201 const toggleNode = new ToggleNode( model.selectedVariabilityProperty, [ { createNode: tandem => new RangeNode( model, sceneModel, { parentContext: 'accordion',