Minor fixes

dy · dy · commit 4efe21ea79a9 · 2017-11-24T17:10:41.000-05:00
diff --git a/src/plot_api/plot_api.js b/src/plot_api/plot_api.js
@@ -2796,6 +2796,7 @@ function makePlotFramework(gd) {
     // right, rather than enter/exit which can muck up the order
     // TODO: sort out all the ordering so we don't have to
     // explicitly delete anything
+    // FIXME: parcoords reuses this object, not the best pattern
     fullLayout._glcontainer = fullLayout._paperdiv.selectAll('.gl-container')
         .data([{}]);
     fullLayout._glcontainer.enter().append('div')
diff --git a/src/plots/plots.js b/src/plots/plots.js
@@ -1332,7 +1332,7 @@ plots.purge = function(gd) {
 
     var fullLayout = gd._fullLayout || {};
     if(fullLayout._glcontainer !== undefined) {
-        fullLayout._glcontainer.selectAll('.gl-canvas').data([]).exit().remove();
+        fullLayout._glcontainer.selectAll('.gl-canvas').remove();
         fullLayout._glcontainer.remove();
         fullLayout._glcanvas = null;
     }

Original file line number	Diff line number	Diff line change
`@@ -1332,7 +1332,7 @@ plots.purge = function(gd) {`
`1332`	`1332`
`1333`	`1333`	`var fullLayout = gd._fullLayout \|\| {};`
`1334`	`1334`	`if(fullLayout._glcontainer !== undefined) {`
`1335`		`- fullLayout._glcontainer.selectAll('.gl-canvas').data([]).exit().remove();`
	`1335`	`+ fullLayout._glcontainer.selectAll('.gl-canvas').remove();`
`1336`	`1336`	`fullLayout._glcontainer.remove();`
`1337`	`1337`	`fullLayout._glcanvas = null;`
`1338`	`1338`	`}`