Changes between Version 2 and Version 3 of MathML/Early_2016_Refactoring


Ignore:
Timestamp:
Mar 31, 2016 5:20:23 AM (6 years ago)
Author:
fred.wang@free.fr
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • MathML/Early_2016_Refactoring

    v2 v3  
     1== Overview ==
     2
    13For the rationale and initial plan see [https://lists.webkit.org/pipermail/webkit-dev/2015-December/027840.html Alex's message] on webkit-dev mailing list. See also [https://github.com/WebKit/webkit/compare/master...fred-wang:mathml Fred's MathML branch] for an overview of the list of commits and bug entries.
    24
    35The idea is now to remove the flexbox dependency (CSS properties & class inheritance) only after having rewritten MathML layout functions from stratch. Hence we no longer need to temporarily "duplicate" flexbox code and this should make review easier. When refactoring renderer classes, we also remove the anonymous renderers (and their associated style) that were created for the sake of the flexbox-based layout.
     6
     7== Layout changes ==
    48
    59[http://www.mathml-association.org/MathMLinHTML5/S3.html#F1 The general box model for MathML renderers] is available on the MathML in HTML5 note where you can also find other rules specific to each element. During this first step, we do not always follow completely the layout suggested in that note. Also for now, we ignore the distinction between ink VS logical metrics as well as classical CSS features like margin, padding, border, absolute position, vertical layout or linebreaking. However we do pay attention to right-to-left rendering of mathematical formulas.
     
    3236
    3337This refactoring will fix many known MathML bugs and open the way for more bug fixes & features (see Bugzilla dependencies for details).
     38
     39== Accessibility Changes ==
     40
     41The following patches impact the a11y code:
     42
     433) [https://bugs.webkit.org/show_bug.cgi?id=153917 RenderMathMLFraction]: The Mac accessibility code seems to need the linethickness relative to the default linethickness (which now depends on the math font).
     44
     454) [https://bugs.webkit.org/show_bug.cgi?id=153918 RenderMathMLScripts]: Some small box shifts happen.
     46
     476) [https://bugs.webkit.org/show_bug.cgi?id=153987 RenderMathMLRoot]: Anonymous wrappers are removed.
     48
     497) [https://bugs.webkit.org/show_bug.cgi?id=155018 RenderMathMLOperator and RenderMathMLToken]: We disable the mac test for mfenced because the render subtree for the anonymous operators has changed.