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

The render result of \dots and \cdots is the same #2315

Closed
YiNiCoo opened this issue Nov 6, 2019 · 3 comments
Closed

The render result of \dots and \cdots is the same #2315

YiNiCoo opened this issue Nov 6, 2019 · 3 comments
Labels
Accepted Issue has been reproduced by MathJax team Fixed Test Needed v3.0

Comments

@YiNiCoo
Copy link

YiNiCoo commented Nov 6, 2019

The render result of \dots and \cdots is the same

@dpvc
Copy link
Member

dpvc commented Nov 6, 2019

To clarify, I'm assuming you mean for x, \dots, y, as they should be \cdots for x+\dots+y. Note that x\dots y does produce \ldots, so it seems not to recognize the situation for commas properly.

@dpvc
Copy link
Member

dpvc commented Nov 8, 2019

This is due to

https://github.com/mathjax/MathJax-src/blob/1e1a30f0a89e52fa7764dbb510763145a4da49f1/ts/input/tex/base/BaseItems.ts#L750-L751

This is because the texClass hasn't been set for the elements yet at this point, and so the default REL class is what is found, so it always uses \cdots. @zorkow, we may need to do another post-filter for this, since the texClass is not set properly until the end of the parsing. Alternatively, the getTexClass() method could be modified to look up the TeX class from the operator dictionary when the node is an mo.

There is a second usage of getTexClass() in BaseMethods.ts that probably needs fixing as well.

@dpvc dpvc self-assigned this Nov 18, 2019
dpvc referenced this issue in mathjax/MathJax-src Nov 18, 2019
… look it up from the operator table. Setting the inherited attributes will set the class, so lookups will no longer be needed. Resolves issue #383.
dpvc referenced this issue in mathjax/MathJax-src Dec 16, 2019
Handle texClass for mo's before inheritance.  #383
@dpvc dpvc removed their assignment Dec 31, 2019
@dpvc
Copy link
Member

dpvc commented Feb 5, 2020

I'm moving this to the main MathJax issue tracker, so it can be found more easily.

@dpvc dpvc transferred this issue from mathjax/MathJax-src Feb 5, 2020
@dpvc dpvc added Accepted Issue has been reproduced by MathJax team Merged Merged into develop branch Test Needed labels Feb 5, 2020
@dpvc dpvc added this to the MathJax v3.0.1 milestone Feb 5, 2020
@dpvc dpvc added Fixed v3.0 and removed Merged Merged into develop branch labels Feb 10, 2020
@dpvc dpvc closed this as completed Feb 10, 2020
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 Test Needed v3.0
Projects
None yet
Development

No branches or pull requests

2 participants