Merge pull request matplotlib#23209 from tfpf/overunder-valign

timhoffm · web-flow · commit f278686f9102 · 2022-06-09T13:23:01.000+02:00
Fix the vertical alignment of overunder symbols.
diff --git a/lib/matplotlib/_mathtext.py b/lib/matplotlib/_mathtext.py
@@ -2200,9 +2200,9 @@ def subsuper(self, s, loc, toks):
                 hlist = HCentered([sub])
                 hlist.hpack(width, 'exactly')
                 vlist.extend([Vbox(0, vgap), hlist])
-                shift = hlist.height + vgap
+                shift = hlist.height + vgap + nucleus.depth
             vlist = Vlist(vlist)
-            vlist.shift_amount = shift + nucleus.depth
+            vlist.shift_amount = shift
             result = Hlist([vlist])
             return [result]
 
diff --git a/lib/matplotlib/tests/baseline_images/test_mathtext/mathtext1_dejavusans_02.png b/lib/matplotlib/tests/baseline_images/test_mathtext/mathtext1_dejavusans_02.png
diff --git a/lib/matplotlib/tests/test_mathtext.py b/lib/matplotlib/tests/test_mathtext.py
@@ -122,6 +122,7 @@
 lightweight_math_tests = [
     r'$\sqrt[ab]{123}$',  # github issue #8665
     r'$x \overset{f}{\rightarrow} \overset{f}{x} \underset{xx}{ff} \overset{xx}{ff} \underset{f}{x} \underset{f}{\leftarrow} x$',  # github issue #18241
+    r'$\sum x\quad\sum^nx\quad\sum_nx\quad\sum_n^nx\quad\prod x\quad\prod^nx\quad\prod_nx\quad\prod_n^nx$',  # GitHub issue 18085
 ]
 
 digits = "0123456789"

Original file line number	Diff line number	Diff line change
`@@ -122,6 +122,7 @@`
`122`	`122`	`lightweight_math_tests = [`
`123`	`123`	`r'$\sqrt[ab]{123}$', # github issue #8665`
`124`	`124`	`r'$x \overset{f}{\rightarrow} \overset{f}{x} \underset{xx}{ff} \overset{xx}{ff} \underset{f}{x} \underset{f}{\leftarrow} x$', # github issue #18241`
	`125`	`+ r'$\sum x\quad\sum^nx\quad\sum_nx\quad\sum_n^nx\quad\prod x\quad\prod^nx\quad\prod_nx\quad\prod_n^nx$', # GitHub issue 18085`
`125`	`126`	`]`
`126`	`127`
`127`	`128`	`digits = "0123456789"`