Skip to content
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

Merged
merged 8 commits into from
Jan 23, 2024

Conversation

bkardell
Copy link
Contributor

@bkardell bkardell commented Sep 27, 2023

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 )

@bkardell bkardell changed the title Draft: Update mathml-related references to use MathML-Core instead of Full… Update mathml-related references to use MathML-Core instead of Full… Oct 4, 2023
@bkardell bkardell assigned bkardell and annevk and unassigned bkardell Oct 4, 2023
@bkardell
Copy link
Contributor Author

bkardell commented Oct 4, 2023

This should be ready now.

Copy link
Member

@annevk annevk left a 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 Show resolved Hide resolved
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>
Copy link
Member

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?

Copy link
Contributor Author

@bkardell bkardell Jan 5, 2024

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?

@emilio
Copy link
Contributor

emilio commented Jan 8, 2024

In general pointing to MathML core sounds good to me.

@bkardell
Copy link
Contributor Author

bkardell commented Jan 8, 2024

@annevk asked

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.

emilio said yes on the Gecko side, @bfgeek Can you give this the 👍 on the Chromium side maybe?

@bfgeek
Copy link
Member

bfgeek commented Jan 8, 2024

lgtm

source Show resolved Hide resolved
source Outdated Show resolved Hide resolved
@annevk annevk merged commit 8b404d4 into whatwg:main Jan 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants