Update TODOs, see #45

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 );

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();
   }

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 );