Update TeXAtom to save the texClass in the property list, so it appears in MathML output. (mathjax/MathJax#2585) #569
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adjusts the TeXAtom node to save the node's TeX class in the property list so that it will be included in the
data-mjx-texclass
attribute when serialized MathML is generated, even if the TeX class is just the defaultORD
class. This allows the serialized MathML to preserve the TeXAtom even when the property was not set explicitly.For example,
f{\left(x\right)}
currently producesand since the outer
<mrow>
does not include a TeX class, the spacing is that ofINNER
when this MathML is read into MathJax (but the spacing isORD
when the TeX input is processed).This PR causes the serialized MathML to be
which gives the same output as the TeX does.
Note that this PR probably will require changes to the test suite, since braced groups will now show
data-mix-texclass="ORD"
explicitly in situations where it didn't before. (E.g.,x^{-4}
).Resolves issue mathjax/MathJax#2585.