From f67ff4d1dcbd4a0a9e5d460ed10020861a8abcae Mon Sep 17 00:00:00 2001 From: Nicolas Riesco Date: Mon, 4 Apr 2016 10:03:38 +0100 Subject: [PATCH] Remove unnecessary step from shapes.drawAll() * Removed call to `.remove()` in `shapes.drawAll()`, since `.remove()` is also call in `shapes.draw()`. * No new test failed after this change. --- src/components/shapes/index.js | 1 - 1 file changed, 1 deletion(-) diff --git a/src/components/shapes/index.js b/src/components/shapes/index.js index a52ad658487..fc6fe8a522a 100644 --- a/src/components/shapes/index.js +++ b/src/components/shapes/index.js @@ -89,7 +89,6 @@ function linearToData(ax) { return ax.type === 'category' ? ax.l2c : ax.l2d; } shapes.drawAll = function(gd) { var fullLayout = gd._fullLayout; - fullLayout._shapelayer.selectAll('path').remove(); for(var i = 0; i < fullLayout.shapes.length; i++) { shapes.draw(gd, i); }