| Summary: | Add support for movablelimits | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Frédéric Wang (:fredw) <fred.wang> | ||||||||||||||||
| Component: | MathML | Assignee: | Frédéric Wang (:fredw) <fred.wang> | ||||||||||||||||
| Status: | RESOLVED FIXED | ||||||||||||||||||
| Severity: | Normal | CC: | alex, commit-queue, dbarton, esprehn+autocc, glenn, kondapallykalyan, mrobinson | ||||||||||||||||
| Priority: | P2 | ||||||||||||||||||
| Version: | WebKit Nightly Build | ||||||||||||||||||
| Hardware: | All | ||||||||||||||||||
| OS: | All | ||||||||||||||||||
| Bug Depends on: | 133845 | ||||||||||||||||||
| Bug Blocks: | 118737 | ||||||||||||||||||
| Attachments: |
|
||||||||||||||||||
Created attachment 274191 [details]
testcase
Created attachment 274192 [details]
testcase
Created attachment 274270 [details]
Patch
Created attachment 277379 [details]
Patch
Created attachment 282053 [details]
Patch
Comment on attachment 282053 [details]
Patch
These changes seem okay. Please confirm tests pass with these updates in place before landing.
Created attachment 283111 [details]
Patch
Committed r202970: <http://trac.webkit.org/changeset/202970> |
Created attachment 274190 [details] Patch (WIP) When the base of an munderover/munder/mover element is an operator with the movablelimits=true property in displaystyle=false ; it should render the same as the equivalent msubsup/msub/sup. After the MathML refactoring, it is easy: we essentially need to make RenderMathMLUnderOver inherits from RenderMathMLScripts. This actually allows to share more code between the two classes. I'm attaching a WIP patch. I plan to add new tests and to test it on mac too tomorrow.