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

CommonHTML doesn't handle moveable limits for inline operators like \lim #970

Closed
dpvc opened this issue Dec 6, 2014 · 5 comments

Comments

Projects
None yet
2 participants
@dpvc
Copy link
Member

commented Dec 6, 2014

Things like \lim_{x\to 0} in in-line mode don't move the limits to super- and subscript positions as they are supposed to.

@dpvc dpvc added the Accepted label Dec 6, 2014

@dpvc dpvc added this to the MathJax 2.5 milestone Dec 6, 2014

dpvc pushed a commit to dpvc/MathJax that referenced this issue Dec 6, 2014

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Dec 6, 2014

I've fixed this in the issue970 branch of my fork of MathJax, and merged it into common-html and CHTML-preview.

@pkra, if you want to update the alpha test site in use at SE, the CHTML-preview branch now has the fixes for the two issue reported on SE. I have not combined and packed the changes, however.

@dpvc

This comment has been minimized.

Copy link
Member Author

commented Dec 7, 2014

=> Merged.

@dpvc dpvc added Merged and removed Ready for Review labels Dec 7, 2014

@dpvc dpvc closed this Dec 7, 2014

@pkra

This comment has been minimized.

Copy link
Member

commented Dec 15, 2014

I've repacked&combined but during smoke-testing I've found a problem with equation labels, see https://rawgit.com/mathjax/MathJax/v2.5-alpha/test/sample-eqrefs.html

@pkra

This comment has been minimized.

Copy link
Member

commented Dec 15, 2014

To be more precise: the first label is broken.

dpvc pushed a commit to dpvc/MathJax that referenced this issue Dec 31, 2014

Davide P. Cervone
Add priority 100 to label reset so that it comes after other post-fil…
…ters (in particular, the CTHML-preview, which can load files). Resolves second issue in mathjax#970.
@dpvc

This comment has been minimized.

Copy link
Member Author

commented Dec 31, 2014

OK, this was due to the order in which the TeX post-filters ran: there is one that handles updating the list of labels and one that handles the CHTML preview. The former needs to run after the latter, in case the CHTML-preview needs to load files (as it does for the first equation, since it loads the CommonHTML config and jax files if they aren't already loaded). So the order matters. I've added priorities to force the order to be correct.

@dpvc dpvc added Fixed v2.5 and removed Merged labels Jan 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.