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

[MathML+useMathMLspacing:true] two adjacent fractions have no space #1626

Closed
pkra opened this issue Sep 28, 2016 · 4 comments
Closed

[MathML+useMathMLspacing:true] two adjacent fractions have no space #1626

pkra opened this issue Sep 28, 2016 · 4 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Available v2.7
Milestone
MathJax v2.7.2

Comments

@pkra
Copy link
Contributor

pkra commented Sep 28, 2016

This was reported on the User Group.

Something like .

<math xmlns="http://www.w3.org/1998/Math/MathML" display="block">
  <mfrac>
    <mi>a</mi>
    <mi>b</mi>
  </mfrac>
  <mfrac>
    <mi>a</mi>
    <mi>b</mi>
  </mfrac>
</math>

will have the fraction lines touch (all outputs).

Up to v2.3, the rendering had some space between the fractions (and without useMathMLspacing that's still the case of course).

Was this an intentional change?
@pkra pkra changed the title [MathML+useMathMLspacing:true] two adjacent fractions have no space with [MathML+useMathMLspacing:true] two adjacent fractions have no space Sep 28, 2016
@dpvc
Copy link
Member

dpvc commented Sep 28, 2016

This was due to the changes for issue #547. A solution would be to remove the && !this.useMMLspacing from line 2581 in the HTML-CSS jax (and the corresponding lines 2482 and 1782 in the CHTML and SVG jax). Alternatively, one could use something other than the TeX nulldelimiterspace value for the useMMLspacing case.

@dpvc dpvc added Ready for Development Accepted Issue has been reproduced by MathJax team and removed Investigate labels Sep 28, 2016
@pkra pkra added this to the A future release milestone Sep 29, 2016
@dpvc dpvc self-assigned this Jun 28, 2017
@dpvc dpvc modified the milestones: A future release, MathJax v2.7.2 Jul 6, 2017
dpvc added a commit to dpvc/MathJax that referenced this issue Jul 7, 2017
@dpvc
Add space between fractions when useMMLspacing is in effect. Resolves…
3c46607 
… issue mathjax#1626.
@dpvc dpvc mentioned this issue Jul 7, 2017
Merged
@dpvc
Copy link
Member

dpvc commented Jul 7, 2017

The issue1626 branch of my fork of MathJax fixes this.

@dpvc dpvc removed their assignment Jul 7, 2017
dpvc added a commit that referenced this issue Jul 10, 2017
@dpvc
Merge pull request #1786 from dpvc/issue1626
70c0d92 
Add space between fractions when useMMLspacing is in effect.  #1626
@dpvc
Copy link
Member

dpvc commented Jul 10, 2017

==> Merged

@dpvc dpvc added Merged Merged into develop branch and removed Ready for Review labels Jul 10, 2017
dpvc added a commit to mathjax/MathJax-test that referenced this issue Jul 11, 2017
@dpvc
Add test for mathjax/MathJax#1626.
860ea9f
@dpvc
Copy link
Member

dpvc commented Jul 11, 2017

==> In testsuite

MathMLToDisplay/Presentation/GeneralLayout/mfrac/issue1626.html

@dpvc dpvc closed this as completed Jul 11, 2017
@dpvc dpvc added Fixed and removed Merged Merged into develop branch labels Jul 18, 2018
@dpvc dpvc added the v2.7 label Jul 18, 2018
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 Available v2.7
Projects
None yet
Milestone
MathJax v2.7.2
Development

No branches or pull requests
2 participants
@pkra @dpvc