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.

Sign up for GitHub

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

Jump to bottom

Stretchy Parallel after Enrichment #2632

Closed
zorkow opened this issue Feb 14, 2021 · 2 comments
Closed

Stretchy Parallel after Enrichment #2632

zorkow opened this issue Feb 14, 2021 · 2 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Needed v3 v3.1
Milestone
3.1.3

Comments

@zorkow
Copy link
Member

zorkow commented Feb 14, 2021

This is the same as SRE #459

\|\hat{f}\|_{X_j},

After enrichment the second parallel is no longer marked as non-stretchy.
The resulting mml structure is similar to what we get from {{\|\hat{f}}\|_{X_j}}, where the second parallel is marked as non-stretchy.
@dpvc
Copy link
Member

dpvc commented Feb 25, 2021

It looks like the second \| is not getting stretchy="false" in the original conversion to MathML, so it is not SRE's fault. I'll look into it further.

@dpvc dpvc added Accepted Issue has been reproduced by MathJax team Investigate v3 labels Feb 25, 2021
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 25, 2021
@dpvc
Fix problem with \| being stretchy when it shouldn't be. (mathjax/Mat…
9d12718 
…hJax#2632)
dpvc added a commit to mathjax/MathJax-src that referenced this issue Feb 25, 2021
@dpvc
Fix problem with \| being stretchy when it shouldn't be. (mathjax/Mat…
349dc0d 
…hJax#2632)
@dpvc dpvc mentioned this issue Feb 25, 2021
Merged
@dpvc
Copy link
Member

dpvc commented Feb 25, 2021

OK, I worked out the problem, and have made a PR for it that explains what was going on.

dpvc added a commit to mathjax/MathJax-src that referenced this issue Mar 12, 2021
@dpvc
Merge pull request #600 from mathjax/issue2632
01a5629 
Fix problem with \| being stretchy when it shouldn't be.  (mathjax/MathJax#2632)
@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Mar 12, 2021
@dpvc dpvc added this to the 3.1.3 milestone Mar 12, 2021
This was referenced Apr 22, 2021
Merged
Merged
Closed
Closed
Closed
@dependabot dependabot bot mentioned this issue Apr 26, 2021
Closed
@dpvc dpvc added Fixed v3.1 and removed Merged Merged into develop branch labels Apr 27, 2021
@dpvc dpvc closed this as completed Apr 27, 2021
This was referenced May 7, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Needed v3 v3.1
Projects
None yet
Milestone
3.1.3
Development

No branches or pull requests
2 participants
@dpvc @zorkow