-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Append ApplyFunction to math functions #960
Conversation
As described at: https://www.w3.org/TR/REC-MathML/chap3_2.html#sec3.2.4 Character U+2061 is the same as ⁡. Ref: https://www.w3schools.com/charsets/ref_html_entities_a.asp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this, and sorry for the delay in reviewing. LGTM, aside from one suggestion for an added comment.
src/functions/operatorname.js
Outdated
@@ -69,7 +69,7 @@ defineFunction({ | |||
identifier.setAttribute("mathvariant", "normal"); | |||
|
|||
const operator = new mathMLTree.MathNode("mo", | |||
[mml.makeText("⁡", "text")]); | |||
[mml.makeText("\u2061", "text")]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe worth a comment that this is <mo>⁡</mo>
as above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do.
Thanks again for the PR! |
It might be nice to have an option to control the output of MathML. I like |
As described at: https://www.w3.org/TR/REC-MathML/chap3_2.html#sec3.2.4
Character U+2061 is the same as . Ref: https://www.w3schools.com/charsets/ref_html_entities_a.asp