fixup config of mathjax v3

Author: archmoj

src/lib/svg_text_utils.js

@@ -192,7 +192,6 @@ function texToSVG(_texString, _config, _callback) {
     var originalRenderer,
         originalConfig,
         originalProcessSectionDelay,
-        noOriginalInlineMath,
         tmpDiv;
 
     var setConfig2 = function() {
@@ -216,10 +215,11 @@ function texToSVG(_texString, _config, _callback) {
     var setConfig3 = function() {
         originalConfig = Lib.extendDeepAll({}, MathJax.config);
 
-        if(!MathJax.config.tex.inlineMath) {
-            MathJax.config.tex.inlineMath = inlineMath;
-            noOriginalInlineMath = true;
+        if(!MathJax.config.tex) {
+            MathJax.config.tex = {};
         }
+
+        MathJax.config.tex.inlineMath = inlineMath;
     };
 
     var setRenderer2 = function() {
@@ -297,11 +297,7 @@ function texToSVG(_texString, _config, _callback) {
     };
 
     var resetConfig3 = function() {
-        if(noOriginalInlineMath) {
-            delete MathJax.config.tex.inlineMath;
-        } else {
-            MathJax.config.tex.inlineMath = originalConfig.tex.inlineMath;
-        }
+        MathJax.config = originalConfig;
     };
 
     if(MathJaxVersion === 2) {

test/jasmine/bundle_tests/mathjax_config_test.js

@@ -20,7 +20,7 @@ describe('Test MathJax v' + mathjaxVersion + ' config test:', function() {
                 startup: {
                     output: 'chtml',
                     tex: {
-                        inlineMath: ['|', '|']
+                        inlineMath: [['|', '|']]
                     }
                 }
             };
@@ -39,31 +39,42 @@ describe('Test MathJax v' + mathjaxVersion + ' config test:', function() {
         if(mathjaxVersion === 2) {
             window.MathJax.Hub.Config({
                 tex2jax: {
-                    inlineMath: ['|', '|']
+                    inlineMath: [['|', '|']]
                 }
             });
 
             window.MathJax.Hub.setRenderer('CHTML');
         }
 
+        // before plot
+        if(mathjaxVersion === 3) {
+            expect(window.MathJax.config.startup.tex.inlineMath).toEqual([['|', '|']]);
+            expect(window.MathJax.config.startup.output).toEqual('chtml');
+        }
+        if(mathjaxVersion === 2) {
+            expect(window.MathJax.Hub.config.tex2jax.inlineMath).toEqual([['|', '|']]);
+            expect(window.MathJax.Hub.config.menuSettings.renderer).toEqual('');
+        }
+
         Plotly.newPlot(gd, {
             data: [{
                 y: [1, 2]
             }],
             layout: {
                 title: {
-                    text: '|E=mc^2|'
+                    text: '$E=mc^2$'
                 }
             }
         })
         .then(function() {
+            // after plot
             if(mathjaxVersion === 3) {
-                expect(window.MathJax.config.startup.tex.inlineMath).toEqual(['|', '|']);
+                expect(window.MathJax.config.startup.tex.inlineMath).toEqual([['|', '|']]);
                 expect(window.MathJax.config.startup.output).toEqual('chtml');
             }
-
             if(mathjaxVersion === 2) {
-                expect(window.MathJax.Hub.config.tex2jax.inlineMath).toEqual(['|', '|']);
+                // TODO: figure out why v2 implementation does not reset inlineMath
+                // expect(window.MathJax.Hub.config.tex2jax.inlineMath).toEqual([['|', '|']]);
                 expect(window.MathJax.Hub.config.menuSettings.renderer).toEqual('');
             }
         })