ensure we create only one modebar-container

Author: antoinerg

src/plot_api/plot_api.js

@@ -3802,6 +3802,7 @@ function makePlotFramework(gd) {
         .classed('gl-container', true);
 
     fullLayout._paperdiv.selectAll('.main-svg').remove();
+    fullLayout._paperdiv.selectAll('.modebar-container').remove();
 
     fullLayout._paper = fullLayout._paperdiv.insert('svg', ':first-child')
         .classed('main-svg', true);

test/jasmine/tests/plot_api_test.js

@@ -2683,6 +2683,23 @@ describe('Test plot api', function() {
             .catch(failTest)
             .then(done);
         });
+
+        it('should only have one modebar-container', function(done) {
+            var data = [{y: [1, 2]}];
+
+            Plotly.plot(gd, data).then(function() {
+                var modebars = document.getElementsByClassName('modebar-container');
+                expect(modebars.length).toBe(1);
+
+                return Plotly.newPlot(gd, data);
+            })
+            .then(function() {
+                var modebars = document.getElementsByClassName('modebar-container');
+                expect(modebars.length).toBe(1);
+            })
+            .catch(failTest)
+            .then(done);
+        });
     });
 
     describe('Plotly.update should', function() {