Bug 161306 – Move direction property into the operator dictionary

• Component: MathML
• Status: RESOLVED WONTFIX

[Frédéric Wang (:fredw)]

Currently MathMLOperatorDictionary has one dictionary indexed by Operator+Form and a list of horizontal operators. We could do like in Gecko and make "horizontal" a boolean property (encoded by a new MathMLOperatorDictionary::Flag), so that it can be directly integrated into the operator dictionary and accessed via MathMLOperatorDictionary::search. We just need to ensure that all the forms have the same direction.

[Frédéric Wang (:fredw)]

(In reply to comment #0)
> We just need to ensure that all the forms have the same direction.

... and we call MathMLOperatorDictionary::search with explicitForm = false

[Frédéric Wang (:fredw)]

Created attachment 287383 [details]
Experimental Patch

[Javier Fernandez]

Comment on attachment 287383 [details]
Experimental Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=287383&action=review

> Source/WebCore/mathml/MathMLOperatorElement.cpp:128
> +        break;

Why we don't return an attribute name for the case of Horizontal ?

[Frédéric Wang (:fredw)]

Comment on attachment 287383 [details]
Experimental Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=287383&action=review

>> Source/WebCore/mathml/MathMLOperatorElement.cpp:128
>> +        break;
> 
> Why we don't return an attribute name for the case of Horizontal ?

Because horizontal is not a MathML attribute. It's just a property that we assign internally to a Unicode character.

[Frédéric Wang (:fredw)]

MathML Core use a separate table: https://w3c.github.io/mathml-core/#dfn-intrinsic-stretch-axis