document why domainProperty.link is the responsibility of WavePacketN…

…umberControl, #165
phetsims · Sep 20, 2021 · 4b98e0f · 4b98e0f
1 parent 8903668
commit 4b98e0f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/js/wavepacket/view/WavePacketNumberControl.js b/js/wavepacket/view/WavePacketNumberControl.js
@@ -64,8 +64,9 @@ class WavePacketNumberControl extends NumberControl {
       [ this.slider.thumbDragListener.isPressedProperty, this.slider.trackDragListener.isPressedProperty ],
       ( thumbIsPressed, trackIsPressed ) => ( thumbIsPressed || trackIsPressed ) );
 
-    // Update the displayed value.
-    //REVIEW: I don't see other dependence on this directly, I presume subtypes do?
+    // Subclasses generally provide options.numberDisplayOptions.numberFormatter which tailors the value display to
+    // the domain. So when the domain changes, this will cause NumberControl tell its NumberDisplay to call
+    // that numberFormatter.
     domainProperty.link( () => this.redrawNumberDisplay() );
   }
 }