- Notifications
You must be signed in to change notification settings - Fork294
Add missing MathML tags for annotations. #183#331
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The sanitizer is built to make sure the output is safe regardless of what the input was. So in order to approve adding these, I need to feel confident that these are safe to add. There's this comment which you wrote that talks about the safety of adding these bits: However, I don't know anything about MathML and can't evaluate that, so I can't approve this. We'll need someone who understands this to yay/nay it preferably with some proof. I'll hunt around and see if I can find someone I know. |
I can't find anyone. I'm going to nix this from the 1.0 milestone. |
@willkg What's the plan on this? Did you ask@jruderman ? |
@fred-wang I didn't find anyone, so I nixed it from the 1.0 milestone. After I got 1.0 out, I stepped off of this project. |
New attempt, rebasing on top of master and restricting to annotations.
https://developer.mozilla.org/en-US/docs/Web/MathML/Element/semantics
https://www.w3.org/TR/MathML/chapter3.html#presm.semantics