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-beta.3: spacing after operatorname #3084

Open
pkra opened this issue Aug 15, 2023 · 3 comments
Open

v4-beta.3: spacing after operatorname #3084

pkra opened this issue Aug 15, 2023 · 3 comments
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 15, 2023

The spacing after an operatorname seems too tight in certain situations (compared to v3 and "real" LaTeX anyway).

E.g.,

<p> Pulling back the Jacobson topology on $\operatorname {Prim}{\mathfrak{T}}$, we obtain a topology on $\operatorname {Spec}{\mathfrak{T}}$ 
</p>
<p>Let $i, j \in \mathbb{I}$ such that $4\lt \operatorname {ord}q_{ii} + \operatorname {ord}q_{jj}$. </p>

This could lead to confusion if font style is the same, e.g., imagine something like $\operatorname{ord}\mathrm{er}$

MathJax v4-beta.3 rendering

image

"real" LaTeX

image

@pkra
Copy link
Contributor Author

pkra commented Aug 15, 2023

Sorry for mixing fonts in the screenshots. This appears to be independent of the fonts.

@dpvc
Copy link
Member

dpvc commented Aug 15, 2023

I have made a PR to fix this.

@pkra
Copy link
Contributor Author

pkra commented Aug 16, 2023

I've tested the patch - thank you!

dpvc added a commit to mathjax/MathJax-src that referenced this issue Aug 17, 2023
Make sure \operatorname stays an OP after enrichment. (mathjax/MathJax#3084)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Aug 17, 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