Fix for { fix for Superscript of tall expression (issue #763) }

[savthegood]

Fix to reflow properly superscript in cases of combo of sub+sup. Move reflow into SupSub so case a_b,right,^x is reflow-ed too. Use margin-bottom to lift superscript and dont touch vertical-align of mq-subsup (so rendering of subs is not messed).

[savthegood]

Test expression I used:
\left(a^{b^{c^{\frac{1}{2}}}}\right)^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b+\left(z^{\frac{1}{2}}\right)^{\frac{1}{2}}+x^{\frac{1}{2}}+x^{\frac{1}{\frac{2}{\frac{3}{\frac{4}{5}}}}}+y_b+y_b^c+y_b+c^2+const

result I got at localhost:

[savthegood]

It all looks good, but there is a downside to this. Superscript is not attached to it's base in any base. So if you enter e.g.

and then change the base to smth with different height: power will not be updated. No way around it, unless reflow of any element would check next element if it's subsup...which might not be the best option.

[savthegood]

And I found a bug(?) :-( :
\left(\frac{c}{\frac{b}{d}}\right)^{\frac{1}{2}}+b

[kschaefe]

Let's ensure that we add your visual tests to the visual tests page. This will help ensure that we don't lose the changes to a future regression.

@laughinghan you can reflow another block, correct? I think that's what needed here. Any tips?

[savthegood]

I've changed the way I calculate needed lift-up for power (see 601946c ).
Test I use

\left(a^{b^{c^{\frac{1}{2}}}}\right)^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b+\left(z^{\frac{1}{2}}\right)^{\frac{1}{2}}+x^{\frac{1}{2}}+x^{\frac{1}{\frac{2}{\frac{3}{\frac{4}{5}}}}}+y_b+y_b^c+y_b+c^2+\left(\frac{c}{\frac{b}{d}}\right)^{\frac{1}{2}}+const

Result I get (I checked FF and Chrome, if you have mac/ie -- check it there too...):

[kschaefe]

Can you please add your test case to the test/visual.html page?

[savthegood]

You mean you want me to add \left(a^{b^{c^{\frac{1}{2}}}}\right)^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b^p+\left(a^{b^{c^{\frac{1}{2}}}}\right)_b+\left(z^{\frac{1}{2}}\right)^{\frac{1}{2}}+x^{\frac{1}{2}}+x^{\frac{1}{\frac{2}{\frac{3}{\frac{4}{5}}}}}+y_b+y_b^c+y_b+c^2+\left(\frac{c}{\frac{b}{d}}\right)^{\frac{1}{2}}+const somewhere in test/visual.html? not sure where exactly should I do it? wouldnt it make more sense smb who wrote that file would add the line in right place?

[savthegood]

@kschaefe ?? pls answer my previous comment.

[kschaefe]

The work here fixes the misalignment @laughinghan noted after we merge #764.

@laughinghan or @stufflebear I have pulled this down using command-line and have updated the visual.html page with @savthegood's monster LaTeX expression.

Do you want me to push to a branch for either of you to review or merge into master?

[manojchandrashekar]

Hi guys - any update on this PR?

[kschaefe]

#841 adds the visual checks for this request. Closing to use the new request.

Dismissing my review because I will merge this. #841 has the requested fix and someone else can approve my change to get that fix in.