fix non-existent font option for SumSymbolNode, #165

phetsims · Sep 20, 2021 · 0578974 · 0578974
1 parent 030c8a3
commit 0578974
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/js/common/view/SumSymbolNode.js b/js/common/view/SumSymbolNode.js
@@ -35,7 +35,7 @@ class SumSymbolNode extends Node {
       // SumSymbolNode options
       integration: false, // true=integration, false=summation
       fontSize: 30,
-      indexFontSize: 12
+      indexFontSize: 15
     }, options );
 
     // The symbol for the type of sum.

diff --git a/js/discrete/view/DiscreteSumEquationNode.js b/js/discrete/view/DiscreteSumEquationNode.js
@@ -54,9 +54,7 @@ class DiscreteSumEquationNode extends Node {
     } );
 
     // Capital sigma, summation symbol
-    const summationNode = new SumSymbolNode( n, 1, numberOfHarmonicsProperty, {
-      font: options.font //REVIEW: I don't see a `font` option in SumSymbolNode, should we be pulling out fontSize?
-    } );
+    const summationNode = new SumSymbolNode( n, 1, numberOfHarmonicsProperty );
 
     // Everything to the right of the summation symbol, set in multilink below
     const rightNode = new RichText( '', {

diff --git a/js/wavepacket/view/WavePacketSumEquationNode.js b/js/wavepacket/view/WavePacketSumEquationNode.js
@@ -48,8 +48,6 @@ class WavePacketSumEquationNode extends Node {
 
     // Capital sigma, summation symbol
     const sumSymbolNode = new SumSymbolNode( FMWSymbols.n, -Infinity, new NumberProperty( Infinity ), {
-      //REVIEW: No `font` option in SumSymbolNode that I can find
-      font: options.font,
       indexFontSize: 15
     } );