8000 ZERO circular dependency by etpinard · Pull Request #2429 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

ZERO circular dependency #2429

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Mar 2, 2018
Merged

ZERO circular dependency #2429

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
bee37dd
move grid module to components/
etpinard Mar 1, 2018
973abbc
merge src/plot_api/register.js into src/registry.js
etpinard Mar 1, 2018
b08c402
fixes #621 - make x/y/z dependent error bar attributes
etpinard Mar 1, 2018
87275af
use errorbars methods via getComponentMethod
etpinard Mar 1, 2018
52b83b3
add apiMethodRegistry in src/registry
etpinard Mar 1, 2018
9fc8832
add Registry.call method + use it to :hocho: circular deps !!!
etpinard Mar 1, 2018
381b6e4
move Plotly.setPlotConfig to plot_api.js
etpinard Mar 1, 2018
ed8d56d
use Registry.traceIs instead of plots.traceIs copy
etpinard Mar 1, 2018
204799c
:hocho: src/plotly.js !!!
etpinard Mar 1, 2018
2c529ec
:lock: down circular dep test at :zero:
etpinard Mar 1, 2018
c6df699
update tests (mostly update ref to spyOn modules)
etpinard Mar 1, 2018
011443e
add `flaky` tag to scattergl visible toggle test
etpinard Mar 1, 2018
9246fda
grab Grid module method using Registry.getComponentMethod
etpinard Mar 2, 2018
d27b3ba
fix typo in test_syntax
etpinard Mar 2, 2018
e05f4f0
rename register_test -> registry_test & improve it!
etpinard Mar 2, 2018
705aed6
replace Registry.call('', []) with Registry.call('', ...[])
etpinard Mar 2, 2018
a851aa5
no need for Registry in plot_api/to_image
etpinard Mar 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
move Plotly.setPlotConfig to plot_api.js
  • Loading branch information
@etpinard
etpinard committed Mar 1, 2018
commit 381b6e402326395b228e6dbb97cedd66c1c6e8d1
3 changes: 1 addition & 2 deletions src/core.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ require('./fonts/mathjax_config');
var Registry = require('./registry');
var register = exports.register = Registry.register;

exports.setPlotConfig = require('./plot_api/set_plot_config');
// expose plot api methods
var plotApi = require('./plot_api');
var methodNames = Object.keys(plotApi);
Expand Down Expand Up @@ -67,7 +66,7 @@ register([
exports.Icons = require('../build/ploticon');

// unofficial 'beta' plot methods, use at your own risk
exports.Plots = Plotly.Plots;
exports.Plots = require('./plots/plots');
exports.Fx = require('./components/fx');
exports.Snapshot = require('./snapshot');
exports.PlotSchema = require('./plot_api/plot_schema');
Expand Down
1 change: 1 addition & 0 deletions src/plot_api/index.js
View file
Open in desktop
10000
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ exports.purge = main.purge;
exports.addFrames = main.addFrames;
exports.deleteFrames = main.deleteFrames;
exports.animate = main.animate;
exports.setPlotConfig = main.setPlotConfig;

exports.toImage = require('./to_image');
exports.validate = require('./validate');
Expand Down
7 changes: 6 additions & 1 deletion src/plot_api/plot_api.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ var Color = require('../components/color');
var xmlnsNamespaces = require('../constants/xmlns_namespaces');
var svgTextUtils = require('../lib/svg_text_utils');

var defaultConfig = require('./plot_config');
var manageArrays = require('./manage_arrays');
var helpers = require('./helpers');
var subroutines = require('./subroutines');
Expand Down Expand Up @@ -447,6 +448,10 @@ exports.plot = function(gd, data, layout, config) {
});
};

exports.setPlotConfig = function setPlotConfig(obj) {
return Lib.extendFlat(defaultConfig, obj);
};

function setBackground(gd, bgColor) {
try {
gd._fullLayout._paper.style('background', bgColor);
Expand All @@ -461,7 +466,7 @@ function opaqueSetBackground(gd, bgColor) {
}

function setPlotContext(gd, config) {
if(!gd._context) gd._context = Lib.extendDeep({}, Plotly.defaultConfig);
if(!gd._context) gd._context = Lib.extendDeep({}, defaultConfig);
var context = gd._context;

var i, keys, key;
Expand Down
24 changes: 0 additions & 24 deletions src/plot_api/set_plot_config.js
View file
Open in desktop

This file was deleted.

0