Ignore the numberDisplay maxWidth for sizing the arrow buttons, see #513

phetsims · Aug 6, 2019 · dfce914 · dfce914
1 parent 199ecaa
commit dfce914
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/js/NumberControl.js b/js/NumberControl.js
@@ -240,12 +240,14 @@ define( require => {
       tandem: options.tandem.createTandem( 'rightArrowButton' )
     }, options.arrowButtonOptions ) );
 
-    // By default, scale the ArrowButtons to have the same height as the NumberDisplay.
+    // By default, scale the ArrowButtons to have the same height as the NumberDisplay, but ignoring
+    // the NumberDisplay's maxWidth (if any)
     if ( !arrowButtonScaleProvided ) {
 
       // Remove the current button scaling so we can determine the desired final scale factor
       leftArrowButton.setScaleMagnitude( 1 );
-      const arrowButtonsScale = numberDisplay.height / leftArrowButton.height;
+      const numberDisplayHeight = numberDisplay.localBounds.height; // ignores maxWidth, if any
+      const arrowButtonsScale = numberDisplayHeight / leftArrowButton.height;
 
       leftArrowButton.setScaleMagnitude( arrowButtonsScale );
       rightArrowButton.setScaleMagnitude( arrowButtonsScale );