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

[SVG] DOM order for \sqrt[2n]{x} #3335

Open
pkra opened this issue Feb 24, 2025 · 3 comments
Open

[SVG] DOM order for \sqrt[2n]{x} #3335

pkra opened this issue Feb 24, 2025 · 3 comments
Labels
Accepted Issue has been reproduced by MathJax team Merged Merged into develop branch v4
Milestone

Comments

@pkra
Copy link
Contributor

pkra commented Feb 24, 2025

While re-testing Speech-Rule-Engine/speech-rule-engine#733 (comment) I noticed a (for me unexpected) DOM order for roots with index, e.g., \sqrt[2n]{x}

While SRE's skeleton structure (now) has index followed by radicand, the SVG output does not.

A naive test indicated that this order is not necessary for the SVG output. If that's the case, it would help to have the "correct"/matching DOM order; it helps with non-visual display in some SSG scenarios.

@dpvc
Copy link
Member

dpvc commented Feb 25, 2025

Yes, the order can be switched. It is not a big change. Do you want me to make you a patch, or a PR that you can apply, or is it something that can wait for the next release?

@dpvc dpvc added Accepted Issue has been reproduced by MathJax team v4 labels Feb 25, 2025
@dpvc dpvc added this to the v4.0 milestone Feb 25, 2025
@pkra
Copy link
Contributor Author

pkra commented Feb 25, 2025

is it something that can wait for the next release?

Of course.

@dpvc
Copy link
Member

dpvc commented Feb 28, 2025

I've made a PR for this, in case you need it before the next release.

dpvc added a commit to mathjax/MathJax-src that referenced this issue Mar 3, 2025
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Issue has been reproduced by MathJax team Merged Merged into develop branch v4
Projects
None yet
Development

No branches or pull requests

2 participants