8000 Clean subplots by alexcjohnson · Pull Request #2227 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

Clean subplots #2227

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
Jan 3, 2018
Merged

Clean subplots #2227

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
687a2e2
refactor calcdata filtering
alexcjohnson Nov 3, 2017
9a18062
perf xaCheater/xaNonCheater test in cartesian axes defaults
alexcjohnson Nov 4, 2017
2c9d90f
lint cartesian layout_defaults
alexcjohnson Nov 4, 2017
3625be4
rework subplot logic - create subplots from traces and components only
alexcjohnson Nov 22, 2017
2de217e
update baselines for improved subplot logic
alexcjohnson Dec 29, 2017
b94ad8e
wider acceptance range for png toimage test
alexcjohnson Dec 29, 2017
c88e8c2
update trans keys
alexcjohnson Dec 22, 2017
26aaffc
fix typo in cameraposition -> camera cleaning
alexcjohnson Nov 30, 2017
276d1d3
updated gl3d "cameraposition" baselines
alexcjohnson Dec 29, 2017
c49007e
:hocho: unused import
alexcjohnson Dec 29, 2017
698561c
:hocho: commented out code
alexcjohnson Dec 30, 2017
8804311
Object.keys -> for in, in plots/plots
alexcjohnson Dec 30, 2017
4786b6e
simplify redundant title removal
alexcjohnson Dec 30, 2017
6e62b35
get rid of unneeded _has(gl2d)
alexcjohnson Dec 30, 2017
0506f88
use _subplots in Axes.list/listIds
alexcjohnson Dec 30, 2017
2aa1545
simplify and speed up a few Axes.list uses
alexcjohnson Dec 30, 2017
44060db
fix annotation calc_autorange change
alexcjohnson Dec 30, 2017
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
get rid of unneeded _has(gl2d)
  • Loading branch information
@alexcjohnson
alexcjohnson committed Dec 30, 2017
commit 6e62b350ce813892b5630fde8e784b434613fd52
2 changes: 1 addition & 1 deletion src/plots/plots.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1371,7 +1371,7 @@ plots.supplyLayoutModuleDefaults = function(layoutIn, layoutOut, fullData, trans
}

// ensure all cartesian axes have at least one subplot
if(layoutOut._has('cartesian') || layoutOut._has('gl2d')) {
if(layoutOut._has('cartesian')) {
Cartesian.finalizeSubplots(layoutIn, layoutOut);
}

Expand Down
0