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

NPM: Add MathJax #8262

Conversation

fneumann
Copy link
Contributor

This PR adds mathjax as NPM dependency.

Usage:

  • Page editor: text content
  • Forum: thread, post, reply
  • Survey: introductory message, final statement
  • Test Questions: question text, feedback, hints
  • Cloze Question: cloze text
  • Kprim Question: answers
  • Matching Question: terms, definitions
  • SC Question: answers
  • MC Question: answers
  • Ordering Question: answers

Wrapped By:

Reasoning:

See Feature Request Streamline Latex Usage

MathJax is currently added as third-party software. It is configured with a URL in the ILIAS administration. That allows ILIAS administrators to adapt their MathJax support, but they need the MathJax CDN or must maintain a local copy themselves. This makes it difficult to compare installations and reproduce errors in the ILIAS test installation. Updates in the case of security problems are the responsibility of local administrators. With MathJax as an npm dependency, this gets much more predictable.

Furthermore, ILIAS components that export HTML content can include MathJax in their exported archives instead of linking it from a CDN.

Maintenance:

MathJax is the de-facto standard for rendering LaTeX on web pages. It is supported by strong partners (IEEE, Elsevier). It is actively developed. In the last month, six pull requests have been merged.

Links:

NPM: https://www.npmjs.com/package/mathjax
GitHub: https://github.com/mathjax
Documentation: https://www.mathjax.org

@klees
Copy link
Member

klees commented Oct 28, 2024

Hi @fneumann,

since this is in Draft-status, I assign you for the time being.

Kind regards!

@fneumann fneumann added the dependencies Pull requests that update a dependency file label Oct 30, 2024
@fneumann fneumann marked this pull request as ready for review October 30, 2024 11:13
@fneumann
Copy link
Contributor Author

Hi @rklees,

I changed the status to "Open" and suggested it for discussion at the JF at 11.11.2024:
https://docu.ilias.de/go/dcl/5316/67_65_11760

Who should be the reviewer of it?

@mjansenDatabay
Copy link
Contributor

Hi @rklees,

I changed the status to "Open" and suggested it for discussion at the JF at 11.11.2024: https://docu.ilias.de/go/dcl/5316/67_65_11760

Who should be the reviewer of it?

No one, you just present this at the upcoming JourFixe.

@matthiaskunkel
Copy link
Member

Is this a dependency of ILIAS 11 / trunk? Or is it still for 10?

@fneumann
Copy link
Contributor Author

fneumann commented Nov 8, 2024

This is for 11 / trunk, since the feature that will use it will be discussed for 11.

@kergomard
Copy link
Contributor

Hi Fred
Just to be sure: You need to update this PR to provide the changes against package_new.js to make sure that this change becomes part of ILIAS 11.

Thank you very much and best,
@kergomard

@fneumann fneumann closed this Dec 3, 2024
@fneumann
Copy link
Contributor Author

fneumann commented Dec 3, 2024

Updated for ILIAS 11 by #8615

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file improvement jour fixe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants