[Webkit-unassigned] [Bug 160509] [meta] Removing/rewriting the mfenced implementation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=160509

Frédéric Wang (:fredw) <fred.wang at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ahmad.saleem792 at gmail.com

--- Comment #3 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
Copying discussion on slack for the record:

ahmad Sep 4th at 2:15 PM
I tried deleting mfenced locally as well to clean-up bit of mathml but it led to whole mfenced not rendering. LOL! (this WPT - https://wpt.fyi/results/mathml/presentation-markup/mrow/legacy-mfenced-element-001.html?label=master&label=experimental&aligned=&q=mfenced

fredw Sep 4th at 2:33 PM
Thanks.Regarding `mfenced`, this would be making it an "unknown MathML element" (https://w3c.github.io/mathml-core/#dfn-unknown-mathml-element) which MathML Core essentially says they behave as an mrow ; unfortunately I'm not sure WebKit implements that latter behavior yet.

fredw Sep 5th at 9:09 AM
I just checked this morning and we have a MathMLUnknownElement class already: https://searchfox.org/wubkat/rev/b36cbce69fddb7da33823f316bd8ead5bebee970/Source/WebCore/mathml/mathtags.in#4Probably making it derive from MathMLRowElement would be enough so that unknown mathml elements can render: https://searchfox.org/wubkat/rev/b36cbce69fddb7da33823f316bd8ead5bebee970/Source/WebCore/mathml/MathMLUnknownElement.h#34And we likely want to remove remove this debug assertion (or override the method in MathMLUnknownElement to create a RenderMathMLRow directly): https://searchfox.org/wubkat/rev/b36cbce69fddb7da33823f316bd8ead5bebee970/Source/WebCore/mathml/MathMLRowElement.cpp#72

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20241010/4f7ad7ae/attachment.htm>