Merge pull request #5514 from plotly/mathjax-config

No longer inject MathJax config by default

Author: archmoj

src/core.js

@@ -8,9 +8,6 @@ require('native-promise-only');
 // inject plot css
 require('../build/plotcss');
 
-// inject default MathJax config
-require('./fonts/mathjax_config')();
-
 // include registry module and expose register method
 var Registry = require('./registry');
 var register = exports.register = Registry.register;

src/fonts/mathjax_config.js

@@ -11,14 +11,14 @@ describe('Test MathJax:', function() {
     // N.B. we have to load MathJax "dynamically" as Karam
     // does not undefined the MathJax's `?config=` parameter.
     //
-    // Eventually, it might be nice to move these tests in the "regular" test
+    // Now with the mathjax_config no longer needed,
+    // it might be nice to move these tests in the "regular" test
     // suites, but to do that we'll need to find a way to remove MathJax from
     // page without breaking things downstream.
     beforeAll(function(done) {
         mathJaxScriptTag = document.createElement('script');
         mathJaxScriptTag.type = 'text/javascript';
         mathJaxScriptTag.onload = function() {
-            require('@src/fonts/mathjax_config')();
             done();
         };
         mathJaxScriptTag.onerror = function() {