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

ui/safe breaks visuals of \longrightleftharpoons #2745

Closed
mhchem opened this issue Jul 25, 2021 · 1 comment
Closed

ui/safe breaks visuals of \longrightleftharpoons #2745

mhchem opened this issue Jul 25, 2021 · 1 comment
Labels
Accepted Issue has been reproduced by MathJax team Fixed v3 v3.2
Milestone

Comments

@mhchem
Copy link

mhchem commented Jul 25, 2021

Please have a look at https://jsbin.com/gapezoravi/1/edit?html,js,console,output, where the upper arrow is smaller than the lower one. This is a strange side-effect of loading ui-safe.

<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width">
  <title>MathJax</title>
  <script>
    window.MathJax = {
      tex: {
        inlineMath: [['$', '$'], ['\\(', '\\)']],
      },
      loader: {load: ['ui/safe']},
    };
  </script>
  <script id="MathJax-script" async src="https://cdn.jsdelivr.net/npm/mathjax@3/es5/tex-svg-full.js"></script>
</head>
<body>
    $\longrightleftharpoons$
</body>
</html>

This should solve mhchem/MathJax-mhchem#21.

@dpvc dpvc added Accepted Issue has been reproduced by MathJax team Investigate v3 labels Jul 26, 2021
@dpvc
Copy link
Member

dpvc commented Jul 26, 2021

I confirm the result you are getting, but am not sure of there cause and will have to investigate further. Apparently the scriptlevel value that is being set on the upper arrow is being removed. The safe extension does limit the allowed values for scriptevel, but it should allow the value 0, which is what it was set to. I'll have to explore what it happening mere fully.

@dpvc dpvc self-assigned this Jul 26, 2021
dpvc added a commit to mathjax/MathJax-src that referenced this issue Jan 31, 2022
@dpvc dpvc added this to the 3.2.1 milestone Jan 31, 2022
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 22, 2022
Make safe extension properly handle scriptlevel of 0. (mathjax/MathJax#2745)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Feb 22, 2022
@dpvc dpvc removed their assignment Jun 1, 2022
@dpvc dpvc added Fixed v3.2 and removed Merged Merged into develop branch labels Jun 1, 2022
@dpvc dpvc closed this as completed Jun 1, 2022
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 Fixed v3 v3.2
Projects
None yet
Development

No branches or pull requests

2 participants