Bug 134020

Summary: Size of radical signs is not updated properly
Product: WebKit Reporter: Frédéric Wang (:fredw) <fred.wang>
Component: MathMLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: alex, bfulgham, rgabor
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 119038, 153987    
Bug Blocks:    
Attachments:
Description Flags
Screenshot none

Description Frédéric Wang (:fredw) 2014-06-18 00:33:54 PDT
Go to the MathML torture test with Linux and check test 13. If you select "Cambria Math" and then go back to the "Default Font", the size of the radical signs is not updated. That seems to be a problem with how the margin-top and stretch size are managed but I could not find exactly where it comes from when working on bug 119038.

BTW, I think there is actually a small "mistake" in RenderMathMLRoot:

radical->style().setMarginTop(Length(radicalTopMargin, Fixed));

should really be radicalWrapper()->... i.e. we should apply the margin to the wrapper, and leave the inner <mo> renderer alone. However, that does not change the problem here.
Comment 1 Frédéric Wang (:fredw) 2014-06-18 00:34:27 PDT
Created attachment 233292 [details]
Screenshot
Comment 2 Frédéric Wang (:fredw) 2014-06-18 00:37:28 PDT
(This also happens if you zoom in and out)
Comment 3 Frédéric Wang (:fredw) 2016-02-05 05:59:02 PST
This bug will be fixed after the RenderMathMLRoot refactoring.
Comment 4 Frédéric Wang (:fredw) 2016-06-17 09:58:16 PDT
I verified that this is fixed by the http://trac.webkit.org/changeset/202168