Skip to content

Commit ee989d9

Browse files
authored
Merge pull request #1063 from plotly/fix-cartesian-funky-layout
Fix dendrogram
2 parents e4617ac + e27fa77 commit ee989d9

File tree

5 files changed

+3900
-3
lines changed

5 files changed

+3900
-3
lines changed

src/plots/cartesian/axes.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1207,7 +1207,7 @@ axes.getSubplots = function(gd, ax) {
12071207
var i, j, sp;
12081208

12091209
// look for subplots in the data
1210-
var data = gd.data || [];
1210+
var data = gd._fullData || gd.data || [];
12111211

12121212
for(i = 0; i < data.length; i++) {
12131213
var trace = data[i];

src/plots/cartesian/index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ exports.rangePlot = function(gd, plotinfo, cdSubplot) {
214214

215215
function makeSubplotData(gd) {
216216
var fullLayout = gd._fullLayout,
217-
subplots = Axes.getSubplots(gd);
217+
subplots = Object.keys(fullLayout._plots);
218218

219219
var subplotData = [],
220220
overlays = [];

src/plots/plots.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ plots.linkSubplots = function(newFullData, newFullLayout, oldFullData, oldFullLa
569569
newSubplots = newFullLayout._plots = {};
570570

571571
var mockGd = {
572-
data: newFullData,
572+
_fullData: newFullData,
573573
_fullLayout: newFullLayout
574574
};
575575

test/image/baselines/dendrogram.png

106 KB
Loading

0 commit comments

Comments
 (0)