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

v4-beta3 SVG inline linebreaking creating invalid markup #3087

Open
pkra opened this issue Aug 17, 2023 · 1 comment
Open

v4-beta3 SVG inline linebreaking creating invalid markup #3087

pkra opened this issue Aug 17, 2023 · 1 comment
Labels
Accepted Issue has been reproduced by MathJax team Merged Merged into develop branch Test Not Needed v4
Milestone

Comments

@pkra
Copy link
Contributor

pkra commented Aug 17, 2023

With inline linebreaking, mathjax generates mjx-linestrut elements inside svg element - which doesn't seem to affect layout but is invalid.

E.g. $a \overset{\text{hello world}}{=} b$ exhibits the problem for me.

@dpvc
Copy link
Member

dpvc commented Aug 17, 2023

In order to handle embellished operators when in-line breaking is allowed, I had to add a function to lay out embellished operators separately from normal ones. It looks like I wrote the CHTML version first, then copied it for SVG and modified it. The mjx-linestrut is needed for CHTML, but not for SVG, and I missed it when adapting the function to the SVG output.

I've made a PR to remove it.

@dpvc dpvc added Ready for Review Accepted Issue has been reproduced by MathJax team Test Not Needed v4 labels Aug 17, 2023
dpvc added a commit to mathjax/MathJax-src that referenced this issue Aug 18, 2023
Remove errant mjx-linestrut from SVG output. (mathjax/MathJax#3087)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Aug 18, 2023
@dpvc dpvc added this to the v4.0 milestone Aug 25, 2023
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 Test Not Needed v4
Projects
None yet
Development

No branches or pull requests

2 participants