Skip to content

Commit

Permalink
Merge pull request #5514 from plotly/mathjax-config
Browse files Browse the repository at this point in the history
No longer inject MathJax config by default
  • Loading branch information
archmoj authored Feb 19, 2021
2 parents e6e2443 + 462d14b commit e65718e
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 26 deletions.
3 changes: 0 additions & 3 deletions src/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
21 changes: 0 additions & 21 deletions src/fonts/mathjax_config.js

This file was deleted.

4 changes: 2 additions & 2 deletions test/jasmine/bundle_tests/mathjax_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand Down

0 comments on commit e65718e

Please sign in to comment.