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

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
use Registry.traceIs instead of plots.traceIs copy
  • Loading branch information
@etpinard
etpinard committed Mar 1, 2018
commit ed8d56daaed2f07e4d48bfd999cc91403a64b1f3
6 changes: 3 additions & 3 deletions src/plots/plots.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1109,7 +1109,7 @@ plots.supplyTraceDefaults = function(traceIn, colorIndex, layout, traceInIndex)
coerce('customdata');
coerce('ids');

if(plots.traceIs(traceOut, 'showLegend')) {
if(Registry.traceIs(traceOut, 'showLegend')) {
coerce('showlegend');
coerce('legendgroup');
}
Expand All @@ -1126,9 +1126,9 @@ plots.supplyTraceDefaults = function(traceIn, colorIndex, layout, traceInIndex)
Lib.coerceHoverinfo(traceIn, traceOut, layout);
}

if(!plots.traceIs(traceOut, 'noOpacity')) coerce('opacity');
if(!Registry.traceIs(traceOut, 'noOpacity')) coerce('opacity');

if(plots.traceIs(traceOut, 'notLegendIsolatable')) {
if(Registry.traceIs(traceOut, 'notLegendIsolatable')) {
// This clears out the legendonly state for traces like carpet that
// cannot be isolated in the legend
traceOut.visible = !!traceOut.visible;
Expand Down
0