sankey - fix issue with large padding

package.json

@@ -57,7 +57,7 @@
   },
   "dependencies": {
     "3d-view": "^2.0.0",
-    "@plotly/d3-sankey": "^0.5.0",
+    "@plotly/d3-sankey": "git://github.com/plotly/d3-sankey.git#6f19390be0da4a81abf6f7f6cfdf5fc8dc73e583",
     "alpha-shape": "^1.0.0",
     "array-range": "^1.0.1",
     "canvas-fit": "^1.5.0",

src/traces/sankey/render.js

@@ -148,6 +148,10 @@ function sankeyModel(layout, d, traceIndex) {
         .links(links)
         .layout(c.sankeyIterations);
 
+    if(sankey.nodePadding() < nodePad) {
+        Lib.warn('node.pad was reduced to ', sankey.nodePadding(), ' to fit within the figure.');
+    }
+
     var node, sankeyNodes = sankey.nodes();
     for(var n = 0; n < sankeyNodes.length; n++) {
         node = sankeyNodes[n];

test/jasmine/tests/sankey_test.js

@@ -771,6 +771,27 @@ describe('sankey tests', function() {
             .then(done);
         });
     });
+
+    it('emits a warning if node.pad is too large', function(done) {
+        var gd = createGraphDiv();
+        var mockCopy = Lib.extendDeep({}, mock);
+
+        var warnings = [];
+        spyOn(Lib, 'warn').and.callFake(function(msg) {
+            warnings.push(msg);
+        });
+        Plotly.plot(gd, mockCopy).then(function() {
+            expect(warnings.length).toEqual(0);
+
+            return Plotly.restyle(gd, 'node.pad', 50);
+        })
+        .then(function() {
+            expect(warnings.length).toEqual(1);
+        })
+        .catch(failTest)
+        .finally(destroyGraphDiv)
+        .then(done);
+    });
 });
 
 function assertLabel(content, style) {