-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Update mathml-related references to use MathML-Core instead of Full… #9804
Conversation
This should be ready now. |
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.
Is there a sense of how much review this new document has gotten?
Also, it would be great if someone from Chromium or Gecko could chime in whether they agree with this change. It seems okay to me, even if the new document is lacking in certain aspects, as that's what appears to be developed still, but I'm very much not in the know.
source
Outdated
<li><dfn data-x-href="https://w3c.github.io/mathml/spec.html#presm_mo">MathML <code>mo</code></dfn> element</li> | ||
<li><dfn data-x-href="https://w3c.github.io/mathml/spec.html#presm_ms">MathML <code>ms</code></dfn> element</li> | ||
<li><dfn data-x-href="https://w3c.github.io/mathml/spec.html#presm_mtext">MathML <code>mtext</code></dfn> element</li> | ||
<li><dfn data-x-href="https://w3c.github.io/mathml-core/#dfn-annotation-xml">MathML <code>annotation-xml</code></dfn> element</li> |
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.
This doesn't seem to lead to an example definition of this element. I guess not each element has its own introduction anymore?
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.
Not in the same way, I suppose - it does describe what it is for and show examples. It's the closest thing I think we have there - Is that problematic?
In general pointing to MathML core sounds good to me. |
@annevk asked
emilio said yes on the Gecko side, @bfgeek Can you give this the 👍 on the Chromium side maybe? |
lgtm |
This is only an update the the relevant links to reference MathML-Core (for browsers) rather than MathML full re: #9795
I'm marking it as draft as it shouldn't be landed until this update to refs in MathML-Core itself are landed
/dom.html ( diff )
/embedded-content-other.html ( diff )
/indices.html ( diff )
/infrastructure.html ( diff )
/parsing.html ( diff )
/references.html ( diff )
/syntax.html ( diff )