308045 - fraction line is shifted

Status: RESOLVED FIXED

(Core :: MathML, defect)

Description

[rbs]

Under certain circumstances, the midline of a fraction is shifted left. See URL
above.

http://groups.google.com/group/netscape.public.mozilla.mathml/browse_frm/thread/8c09e23f81671087/f7e885d4d3facb78?lnk=st&q=mfrac+latency%3F+--+midline+shifted&rnum=1&hl=en#f7e885d4d3facb78

Comment 1

[rbs]

Attachment: fix

Following the general sweep of bug 306543 to improve the inter-spacing, I got
feedback from wikipedia folks that it did the trick for them. I am now turning
attention to the edge cases. As far I can tell, this patch covers all the
remaining tags that need special treatments. It covers <mfrac>, <msqrt>,
<mfenced>, which contain things like the fraction line, the sqrt glyph and sqrt
bar, or fences and separators that are part of the flow without having frames.

[Note: I am aware of the very particular bug 118743. It is quite different and
is not my radar in this round.]

Comment 2

[rbs]

I would also like in the course of checkin this patch, to update the spacing
table in nsMathMLContainerFrame.cpp as follows:
                                       >HERE<
   /*           Ord  OpOrd OpInv OpUsr Inner Italic Upright */
-  /*Ord  */   {0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00},
+  /*Ord  */   {0x00, 0x00, 0x00, 0x01, 0x01, 0x00, 0x00},
   /*OpOrd*/   {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
   /*OpInv*/   {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
   /*OpUsr*/   {0x01, 0x00, 0x00, 0x01, 0x01, 0x01, 0x01},
-  /*Inner*/   {0x00, 0x00, 0x00, 0x01, 0x01, 0x00, 0x00},
+  /*Inner*/   {0x01, 0x00, 0x00, 0x01, 0x01, 0x01, 0x01}, <<HERE
-  /*Italic*/  {0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x01},
+  /*Italic*/  {0x00, 0x00, 0x00, 0x01, 0x01, 0x00, 0x01},
-  /*Upright*/ {0x00, 0x00, 0x00, 0x01, 0x00, 0x01, 0x01}
+  /*Upright*/ {0x00, 0x00, 0x00, 0x01, 0x01, 0x01, 0x01}

<mfrac> is of type 'Inner' on that table. So the change updates its row (resp.
column) to also put a thin space before (resp. after) a fraction if it is
preceded (resp. followed) by an Ord, Italic, or Upright, MathML frame type.

Comment 3

[rbs]

roc, I will be travelling next week and won't have a chance to follow-up. Would
you mind r/sr now so that I can request branch approval?

Comment 4

[rbs]

Comment on attachment 195683 [details] [diff] [review]
fix

Remaining piece to go with bug 308045.

Comment 5

[rbs]

Checked in the trunk.

Comment 6

[rbs]

Checked in the 1.8 branch.