mutate translation options after bounds are set, phetsims/scenery#1340

phetsims · Jun 28, 2022 · 4111d0f · 4111d0f
1 parent f20ca26
commit 4111d0f
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/js/NumberPicker.js b/js/NumberPicker.js
@@ -156,6 +156,9 @@ class NumberPicker extends AccessibleNumberSpinner( Node, 0 ) {
     options.keyboardStep = keyboardStep;
     options.shiftKeyboardStep = keyboardStep;
 
+    const boundsRequiredOptionKeys = _.pick( options, Node.REQUIRES_BOUNDS_OPTION_KEYS );
+    options = _.omit( options, Node.REQUIRES_BOUNDS_OPTION_KEYS );
+
     super( options );
 
     //------------------------------------------------------------
@@ -415,6 +418,9 @@ class NumberPicker extends AccessibleNumberSpinner( Node, 0 ) {
       tandem: options.tandem.createTandem( 'valueProperty' )
     } );
 
+    // Mutate options that require bounds after we have children
+    this.mutate( boundsRequiredOptionKeys );
+
     // @private
     this.disposeNumberPicker = () => {