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

Add support for MathJax 3, upgrade vendored MathJax to 3.2.2 #740

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ManifoldFR
Copy link
Contributor

@ManifoldFR ManifoldFR commented Jan 22, 2025

Follow up to #737 and #738

This pull request updates the vendored files for MathJax to the latest release, 3.2.2.

The previous version that was shipped with jrl-cmakemodules was from an old 2.x from over a decade ago. MathJax 3.x is a full rewrite which is much faster than 2.x (partly as a response to the development of KaTeX which was leaner and faster).

Major changes

  • Upgraded vendored version of MathJax to 3.2.2 (June 2022)
  • Added support for Doxygen option MATHJAX_VERSION, which defaults to MathJax_3 (for Doxygen>=1.9.2. Other users get MathJax 2.)
  • In case MathJax version is set to 3, default value of MATHJAX_RELPATH is set to
    ${PROJECT_JRL_CMAKE_MODULE_DIR}/doxygen/MathJax (i.e. our vendored MathJax). If the user desires MathJax 2, they can set the path themselves, or leave it unset to let Doxygen inject the MathJax 2.x CDN instead.
  • Added the entirety of doxygen/MathJax to the pre-commit exclusion rules.

@ManifoldFR ManifoldFR marked this pull request as draft January 24, 2025 14:55
@ManifoldFR ManifoldFR force-pushed the topic/doxygen-mathjax-2025 branch 4 times, most recently from be473e5 to 3ed3531 Compare January 24, 2025 16:02
@ManifoldFR ManifoldFR force-pushed the topic/doxygen-mathjax-2025 branch from 3ed3531 to 42213fd Compare January 24, 2025 16:04
@ManifoldFR ManifoldFR changed the title Upgrade vendored MathJax to 3.2.2 release Add support for MathJax 3, upgrade vendored MathJax to 3.2.2 Jan 24, 2025
@ManifoldFR ManifoldFR marked this pull request as ready for review January 24, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants