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

Frédéric Wang (:fredw)
Reported 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.
Attachments
Screenshot (4.16 KB, image/png)
2014-06-18 00:34 PDT, Frédéric Wang (:fredw) 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Frédéric Wang (:fredw)
Comment 1 2014-06-18 00:34:27 PDT
Created attachment 233292 [details] Screenshot
Frédéric Wang (:fredw)
Comment 2 2014-06-18 00:37:28 PDT
(This also happens if you zoom in and out)
Frédéric Wang (:fredw)
Comment 3 2016-02-05 05:59:02 PST
This bug will be fixed after the RenderMathMLRoot refactoring.
Frédéric Wang (:fredw)
Comment 4 2016-06-17 09:58:16 PDT
I verified that this is fixed by the http://trac.webkit.org/changeset/202168
Note You need to log in before you can comment on or make changes to this bug.