diff --git a/cms/djangoapps/pipeline_js/js/xmodule.js b/cms/djangoapps/pipeline_js/js/xmodule.js index f27ead886d..3acc0958e0 100644 --- a/cms/djangoapps/pipeline_js/js/xmodule.js +++ b/cms/djangoapps/pipeline_js/js/xmodule.js @@ -18,7 +18,7 @@ define( window._ = _; $script( - 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js' + + '//cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js' + '?config=TeX-MML-AM_SVG&delayStartupUntil=configured', 'mathjax', function() { diff --git a/cms/static/cms/js/require-config.js b/cms/static/cms/js/require-config.js index d043708c5a..4b7e60f2ac 100644 --- a/cms/static/cms/js/require-config.js +++ b/cms/static/cms/js/require-config.js @@ -55,11 +55,8 @@ // the option as displayed in the context menu to false. // When upgrading to 2.6, check if this variable name changed. window.MathJax = { - menuSettings: { - CHTMLpreview: false - } + menuSettings: {CHTMLpreview: false} }; - }; defineDependency('jQuery', 'jquery'); @@ -134,7 +131,7 @@ 'lang_edx': 'js/src/lang_edx', // externally hosted files - mathjax: 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len + mathjax: '//cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len 'youtube': [ // youtube URL does not end in '.js'. We add '?noext' to the path so // that require.js adds the '.js' to the query component of the URL, diff --git a/cms/static/cms/js/spec/main.js b/cms/static/cms/js/spec/main.js index f5aa089b04..1f32ced8d6 100644 --- a/cms/static/cms/js/spec/main.js +++ b/cms/static/cms/js/spec/main.js @@ -69,7 +69,7 @@ 'domReady': 'xmodule_js/common_static/js/vendor/domReady', 'URI': 'xmodule_js/common_static/js/vendor/URI.min', 'mock-ajax': 'xmodule_js/common_static/js/vendor/mock-ajax', - mathjax: 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len + mathjax: '//cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len 'youtube': '//www.youtube.com/player_api?noext', 'js/src/ajax_prefix': 'xmodule_js/common_static/js/src/ajax_prefix', 'js/spec/test_utils': 'js/spec/test_utils' diff --git a/cms/static/cms/js/spec/main_squire.js b/cms/static/cms/js/spec/main_squire.js index 34ff4ace8f..936aa39055 100644 --- a/cms/static/cms/js/spec/main_squire.js +++ b/cms/static/cms/js/spec/main_squire.js @@ -48,7 +48,7 @@ 'draggabilly': 'xmodule_js/common_static/js/vendor/draggabilly', 'domReady': 'xmodule_js/common_static/js/vendor/domReady', 'URI': 'xmodule_js/common_static/js/vendor/URI.min', - mathjax: 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len + mathjax: '//cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len 'youtube': '//www.youtube.com/player_api?noext', 'js/src/ajax_prefix': 'xmodule_js/common_static/js/src/ajax_prefix' }, diff --git a/common/static/common/js/discussion/mathjax_include.js b/common/static/common/js/discussion/mathjax_include.js index f5d18e8077..a2e8f36898 100644 --- a/common/static/common/js/discussion/mathjax_include.js +++ b/common/static/common/js/discussion/mathjax_include.js @@ -42,15 +42,6 @@ if (typeof MathJax === 'undefined') { }); }; }; - - // Automatic loading of Mathjax accessibility files - window.MathJax = { - menuSettings: { - collapsible: true, - autocollapse: true, - explorer: true - } - }; - vendorScript.src = 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js?config=TeX-MML-AM_SVG'; + vendorScript.src = 'https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_SVG'; document.body.appendChild(vendorScript); } diff --git a/common/static/common/js/discussion/utils.js b/common/static/common/js/discussion/utils.js index a4bcbd4d31..576c062ec4 100644 --- a/common/static/common/js/discussion/utils.js +++ b/common/static/common/js/discussion/utils.js @@ -482,10 +482,10 @@ this.postMathJaxProcessor(this.markdownWithHighlight(element.text())) ); + this.typesetMathJax(element); }; DiscussionUtil.typesetMathJax = function(element) { - if (typeof MathJax !== 'undefined' && MathJax !== null) { MathJax.Hub.Queue(['Typeset', MathJax.Hub, element[0]]); } diff --git a/common/static/common/js/discussion/views/discussion_thread_show_view.js b/common/static/common/js/discussion/views/discussion_thread_show_view.js index 51ca3aa3c1..f9b70fe672 100644 --- a/common/static/common/js/discussion/views/discussion_thread_show_view.js +++ b/common/static/common/js/discussion/views/discussion_thread_show_view.js @@ -59,12 +59,13 @@ this.renderAttrs(); this.$('span.timeago').timeago(); this.convertMath(); + this.$('.post-body'); + this.$('h1,h3'); return this; }; DiscussionThreadShowView.prototype.convertMath = function() { DiscussionUtil.convertMath(this.$('.post-body')); - DiscussionUtil.typesetMathJax(this.$('.post-body')); }; DiscussionThreadShowView.prototype.edit = function(event) { diff --git a/common/static/common/js/discussion/views/discussion_thread_view.js b/common/static/common/js/discussion/views/discussion_thread_view.js index 7978eb7486..3346575328 100644 --- a/common/static/common/js/discussion/views/discussion_thread_view.js +++ b/common/static/common/js/discussion/views/discussion_thread_view.js @@ -317,10 +317,7 @@ if (options.focusAddedResponse) { this.focusToTheAddedResponse(view.el); } - - // Typeset the response when initially loaded for any forum - DiscussionUtil.typesetMathJax(view.$el); - return view; + DiscussionUtil.typesetMathJax(view.$el.find('.js-response-list')); }; DiscussionThreadView.prototype.renderAddResponseButton = function() { @@ -348,7 +345,7 @@ }; DiscussionThreadView.prototype.submitComment = function(event) { - var body, comment, url, view; + var body, comment, url; event.preventDefault(); url = this.model.urlFor('reply'); body = this.getWmdContent('reply-body'); @@ -368,7 +365,7 @@ user_id: window.user.get('id') }); comment.set('thread', this.model.get('thread')); - view = this.renderResponseToList(comment, '.js-response-list', { + this.renderResponseToList(comment, '.js-response-list', { focusAddedResponse: true }); this.model.addComment(); @@ -382,9 +379,8 @@ body: body }, success: function(data) { - comment.updateInfo(data.annotated_content_info); - comment.set(data.content); - DiscussionUtil.typesetMathJax(view.$el.find('.response-body')); + comment.updateInfo(data.annotated_content_info); + return comment.set(data.content); } }); }; diff --git a/common/static/common/js/discussion/views/response_comment_show_view.js b/common/static/common/js/discussion/views/response_comment_show_view.js index 99f425036d..da66d28315 100644 --- a/common/static/common/js/discussion/views/response_comment_show_view.js +++ b/common/static/common/js/discussion/views/response_comment_show_view.js @@ -72,8 +72,6 @@ ResponseCommentShowView.prototype.convertMath = function() { DiscussionUtil.convertMath(this.$el.find('.response-body')); - DiscussionUtil.typesetMathJax(this.$el.find('.response-body')); - }; ResponseCommentShowView.prototype._delete = function(event) { diff --git a/common/static/common/js/discussion/views/thread_response_view.js b/common/static/common/js/discussion/views/thread_response_view.js index e3f4dd3b6e..3fb47e9650 100644 --- a/common/static/common/js/discussion/views/thread_response_view.js +++ b/common/static/common/js/discussion/views/thread_response_view.js @@ -82,9 +82,7 @@ ThreadResponseView.prototype.render = function() { this.$el.addClass('response_' + this.model.get('id')); - - // Setting the HTML through utils to cope for tag-less text - edx.HtmlUtils.setHtml(this.$el, edx.HtmlUtils.HTML(this.renderTemplate())); + this.$el.html(this.renderTemplate()); this.delegateEvents(); this.renderShowView(); this.renderAttrs(); @@ -311,7 +309,6 @@ event.preventDefault(); this.createShowView(); this.renderShowView(); - DiscussionUtil.typesetMathJax(this.$el.find('.response-body')); return this.showCommentForm(); }; @@ -345,7 +342,6 @@ }); self.createShowView(); self.renderShowView(); - DiscussionUtil.typesetMathJax(self.$el.find('.response-body')); return self.showCommentForm(); } }); diff --git a/common/templates/mathjax_include.html b/common/templates/mathjax_include.html index 1f46d5098b..cbae907011 100644 --- a/common/templates/mathjax_include.html +++ b/common/templates/mathjax_include.html @@ -75,20 +75,9 @@ } - - - + %endif diff --git a/lms/static/lms/js/spec/main.js b/lms/static/lms/js/spec/main.js index bf9a337837..a845786974 100644 --- a/lms/static/lms/js/spec/main.js +++ b/lms/static/lms/js/spec/main.js @@ -56,7 +56,7 @@ 'squire': 'common/js/vendor/Squire', 'jasmine-imagediff': 'xmodule_js/common_static/js/vendor/jasmine-imagediff', 'domReady': 'xmodule_js/common_static/js/vendor/domReady', - mathjax: 'https://cdn.jsdelivr.net/npm/mathjax@2.7.5/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len + mathjax: '//cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.1/MathJax.js?config=TeX-MML-AM_SVG&delayStartupUntil=configured', // eslint-disable-line max-len 'youtube': '//www.youtube.com/player_api?noext', 'js/src/ajax_prefix': 'xmodule_js/common_static/js/src/ajax_prefix', 'js/instructor_dashboard/student_admin': 'js/instructor_dashboard/student_admin',