Use MathJax 2.4 from cdnjs

Description

This is a VERY small review from Ops perspective: just want ops to weigh in on if the hosting approach for Mathjax files is reasonable.

edx-platform is currently using MathJax 2.2 that's loaded from an S3 bucket (https://github.com/edx/edx-platform/blob/cc6e63dd222902a4cdc7ddc6655fbbddd2b16282/common/templates/mathjax_include.html). One of our clients that has built an external UI based on jquery-xblock has been experiencing weird rendering issues in IE, which we tracked down to a bug in MathJax, most probably related to https://github.com/mathjax/MathJax/issues/585. Switching to MathJax 2.4 in xblock-discussion (https://github.com/edx-solutions/xblock-discussion/pull/24) fixed the problem for us.

I am not sure what the reason for hosting MathJax on S3 is, although it makes sense to host it externally since it's a huge dependency as far as JS libraries go. There is also a vendored version at https://github.com/edx/edx-platform/tree/cfae1cdf62fb74307b88bd091cd4ba927af6b1fc/common/static/js/vendor/mathjax-MathJax-c9db6ac, but that's an old 2.0 version that only appears to be used in xmodule JS specs (https://github.com/edx/edx-platform/blob/master/common/lib/xmodule/xmodule/js/js_test.yml#L54). I would be happy to try and port the specs to use the externally hosted version and removed the vendored MathJax lib if there is interest.

Any kind of feedback would be much appreciated.

Status

Assignee

Adam Palay

Reporter

Edx Admin [Administrator]

Labels

Contributor Name

Matjaz Gregoric

Repo

edx/edx-platform

Customer

Epic Link

None

OSCM Assignee

None

Platform Map Area (Levels 1 & 2)

None

Platform Map Area (Levels 3 & 4)

None

Priority

Unset
Configure