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

missing MathEvents extension #85

Closed
pkra opened this issue Mar 30, 2015 · 3 comments · Fixed by #71
Closed

missing MathEvents extension #85

pkra opened this issue Mar 30, 2015 · 3 comments · Fixed by #71
Labels
Accepted Fixed
Milestone
2.5-relase integr...

Comments

@pkra
Copy link
Contributor

pkra commented Mar 30, 2015

It looks like #82 and the re-opened #66 are due to the MathEvents extension having been stripped from the slimmed-down copy of core MathJax.

@dpvc should I just add it back to the upstream mathjax-node branch or is this worth investigating?
This was referenced Mar 30, 2015
Closed
Closed
@dpvc
Copy link
Member

dpvc commented Mar 30, 2015

MathEvents.js is called in by all the output jax as they need it to attach the event handlers for things like the context menu and zoom clicking to the math output. These handlers are in the MathEvents.js file. Without it, you not only get the errors that you are seeing, but also have a 15 second delay as MathJax waits for the unavailable file to load.

I can add some code to mj-page and mj-single to compensate.

dpvc added a commit that referenced this issue Mar 30, 2015
@dpvc
Compensate for removed MathEvents.js. Resolves issues #85, #66, and #82.
0b4b10f
@dpvc
Copy link
Member

dpvc commented Mar 30, 2015

I have pushed a change that should resolve the problem.

@pkra
Copy link
Contributor Author

pkra commented Apr 1, 2015

This hasn't been merged into develop?

This was referenced Apr 1, 2015
Merged
Merged
@pkra pkra closed this as completed in #71 Apr 1, 2015
@dpvc dpvc added this to the 2.5-relase integration milestone Apr 27, 2015
physikerwelt pushed a commit to physikerwelt/MathJax-node that referenced this issue Nov 5, 2015
@dpvc @physikerwelt
Compensate for removed MathEvents.js. Resolves issues mathjax#85, mat…
e35b35d 
…hjax#66, and mathjax#82.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Accepted Fixed
Projects
None yet
Milestone
2.5-relase integration
Development

Successfully merging a pull request may close this issue.

Release 0.3 mathjax/MathJax-node
2 participants
@pkra @dpvc