fixup: don't support array value in sankey hoverlabel settings

- see 4d3ce92#r115788926
  for rationale

Author: etpinard

src/traces/sankey/plot.js

@@ -12,6 +12,7 @@ var d3 = require('d3');
 var render = require('./render');
 var Fx = require('../../components/fx');
 var Color = require('../../components/color');
+var Lib = require('../../lib');
 
 function renderableValuePresent(d) {return d !== '';}
 
@@ -106,6 +107,13 @@ function linkNonHoveredStyle(d, sankey, visitNodes, sankeyLink) {
     }
 }
 
+// does not support array values for now
+function castHoverOption(trace, attr) {
+    var labelOpts = trace.hoverlabel || {};
+    var val = Lib.nestedProperty(labelOpts, attr).get();
+    return Array.isArray(val) ? false : val;
+}
+
 module.exports = function plot(gd, calcData) {
 
     var fullLayout = gd._fullLayout;
@@ -125,7 +133,6 @@ module.exports = function plot(gd, calcData) {
 
     var linkHoverFollow = function(element, d) {
         var trace = gd._fullData[d.traceId];
-        var ptNumber = d.originalIndex;
         var boundingBox = element.getBoundingClientRect();
         var hoverCenterX = boundingBox.left + boundingBox.width / 2;
         var hoverCenterY = boundingBox.top + boundingBox.height / 2;
@@ -139,11 +146,11 @@ module.exports = function plot(gd, calcData) {
                 ['Source:', d.link.source.label].join(' '),
                 ['Target:', d.link.target.label].join(' ')
             ].filter(renderableValuePresent).join('<br>'),
-            color: Fx.castHoverOption(trace, ptNumber, 'bgcolor') || Color.addOpacity(d.tinyColorHue, 1),
-            borderColor: Fx.castHoverOption(trace, ptNumber, 'bordercolor'),
-            fontFamily: Fx.castHoverOption(trace, ptNumber, 'font.family'),
-            fontSize: Fx.castHoverOption(trace, ptNumber, 'font.size'),
-            fontColor: Fx.castHoverOption(trace, ptNumber, 'font.color'),
+            color: castHoverOption(trace, 'bgcolor') || Color.addOpacity(d.tinyColorHue, 1),
+            borderColor: castHoverOption(trace, 'bordercolor'),
+            fontFamily: castHoverOption(trace, 'font.family'),
+            fontSize: castHoverOption(trace, 'font.size'),
+            fontColor: castHoverOption(trace, 'font.color'),
             idealAlign: d3.event.x < hoverCenterX ? 'right' : 'left'
         }, {
             container: fullLayout._hoverlayer.node(),
@@ -177,7 +184,6 @@ module.exports = function plot(gd, calcData) {
 
     var nodeHoverFollow = function(element, d) {
         var trace = gd._fullData[d.traceId];
-        var ptNumber = d.originalIndex;
         var nodeRect = d3.select(element).select('.nodeRect');
         var boundingBox = nodeRect.node().getBoundingClientRect();
         var hoverCenterX0 = boundingBox.left - 2;
@@ -194,11 +200,11 @@ module.exports = function plot(gd, calcData) {
                 ['Incoming flow count:', d.node.targetLinks.length].join(' '),
                 ['Outgoing flow count:', d.node.sourceLinks.length].join(' ')
             ].filter(renderableValuePresent).join('<br>'),
-            color: Fx.castHoverOption(trace, ptNumber, 'bgcolor') || d.tinyColorHue,
-            borderColor: Fx.castHoverOption(trace, ptNumber, 'bordercolor'),
-            fontFamily: Fx.castHoverOption(trace, ptNumber, 'font.family'),
-            fontSize: Fx.castHoverOption(trace, ptNumber, 'font.size'),
-            fontColor: Fx.castHoverOption(trace, ptNumber, 'font.color'),
+            color: castHoverOption(trace, 'bgcolor') || d.tinyColorHue,
+            borderColor: castHoverOption(trace, 'bordercolor'),
+            fontFamily: castHoverOption(trace, 'font.family'),
+            fontSize: castHoverOption(trace, 'font.size'),
+            fontColor: castHoverOption(trace, 'font.color'),
             idealAlign: 'left'
         }, {
             container: fullLayout._hoverlayer.node(),