diff --git a/js/common/model/CAVObjectType.ts b/js/common/model/CAVObjectType.ts index d93077c7..b6103a65 100644 --- a/js/common/model/CAVObjectType.ts +++ b/js/common/model/CAVObjectType.ts @@ -16,7 +16,7 @@ export default class CAVObjectType extends EnumerationValue { public static readonly DATA_POINT = new CAVObjectType( 1 ); public static readonly SOCCER_BALL = new CAVObjectType( 0.3 ); - // TODO: This term is confusing because sometimes these are rendered as 'x' marks + // TODO: This term is confusing because sometimes these are rendered as 'x' marks. Check all usages--sometimes it is in a dot plot and sometimes a line plot public static readonly DOT = new CAVObjectType( 0.127 ); public static readonly enumeration = new Enumeration( CAVObjectType ); diff --git a/js/common/model/SoccerModel.ts b/js/common/model/SoccerModel.ts index 0edeb718..bf35a366 100644 --- a/js/common/model/SoccerModel.ts +++ b/js/common/model/SoccerModel.ts @@ -311,11 +311,7 @@ export default class SoccerModel extends CAVModel { this.timeWhenLastBallWasKickedProperty.reset(); this.ballPlayerMap.clear(); this.soccerPlayerGroup.clear(); - - // TODO: SR: super.clearData() is called multiple times from reset. - // SR: This could be split up, but in my opinion it is clear and safe. Maybe best to call it twice - super.clearData(); - + super.clearData(); // NOTE: clearData is called twice, but that's not a problem this.populateSoccerPlayerGroup(); this.nextBallToKickProperty.value = this.createBall(); } diff --git a/js/common/view/CardNode.ts b/js/common/view/CardNode.ts index e9bf474e..a849cdb5 100644 --- a/js/common/view/CardNode.ts +++ b/js/common/view/CardNode.ts @@ -84,7 +84,7 @@ export default class CardNode extends Node { start: () => { this.moveToFront(); }, - // TODO-UX: This emits for dragging the leftmost card to the left + // TODO-UX: This emits for dragging the leftmost card to the left, see https://github.com/phetsims/center-and-variability/issues/150 drag: ( event, listener ) => this.dragDistanceEmitter.emit( Math.abs( listener.modelDelta.x ) ) } ); this.addInputListener( this.dragListener );