add cone-specific hoverinfo flags

Author: etpinard

Diff for: src/plots/gl3d/scene.js

@@ -71,28 +71,53 @@ function render(scene) {
         trace = lastPicked.data;
         var ptNumber = selection.index;
         var hoverinfo = Fx.castHoverinfo(trace, scene.fullLayout, ptNumber);
+        var hoverinfoParts = hoverinfo.split('+');
+        var isHoverinfoAll = hoverinfo === 'all';
 
-        var xVal = formatter('xaxis', selection.traceCoordinate[0]),
-            yVal = formatter('yaxis', selection.traceCoordinate[1]),
-            zVal = formatter('zaxis', selection.traceCoordinate[2]);
+        var xVal = formatter('xaxis', selection.traceCoordinate[0]);
+        var yVal = formatter('yaxis', selection.traceCoordinate[1]);
+        var zVal = formatter('zaxis', selection.traceCoordinate[2]);
 
-        if(hoverinfo !== 'all') {
-            var hoverinfoParts = hoverinfo.split('+');
+        if(!isHoverinfoAll) {
             if(hoverinfoParts.indexOf('x') === -1) xVal = undefined;
             if(hoverinfoParts.indexOf('y') === -1) yVal = undefined;
             if(hoverinfoParts.indexOf('z') === -1) zVal = undefined;
             if(hoverinfoParts.indexOf('text') === -1) selection.textLabel = undefined;
             if(hoverinfoParts.indexOf('name') === -1) lastPicked.name = undefined;
         }
 
+        var tx;
+
+        if(trace.type === 'cone') {
+            var coneTx = [];
+            if(isHoverinfoAll || hoverinfoParts.indexOf('u') !== -1) {
+                coneTx.push('u: ' + formatter('xaxis', selection.traceCoordinate[3]));
+            }
+            if(isHoverinfoAll || hoverinfoParts.indexOf('v') !== -1) {
+                coneTx.push('v: ' + formatter('yaxis', selection.traceCoordinate[4]));
+            }
+            if(isHoverinfoAll || hoverinfoParts.indexOf('w') !== -1) {
+                coneTx.push('w: ' + formatter('zaxis', selection.traceCoordinate[5]));
+            }
+            if(isHoverinfoAll || hoverinfoParts.indexOf('norm') !== -1) {
+                coneTx.push('norm: ' + selection.traceCoordinate[6].toPrecision(3));
+            }
+            if(selection.textLabel) {
+                coneTx.push(selection.textLabel);
+            }
+            tx = coneTx.join('<br>');
+        } else {
+            tx = selection.textLabel;
+        }
+
         if(scene.fullSceneLayout.hovermode) {
             Fx.loneHover({
                 x: (0.5 + 0.5 * pdata[0] / pdata[3]) * width,
                 y: (0.5 - 0.5 * pdata[1] / pdata[3]) * height,
                 xLabel: xVal,
                 yLabel: yVal,
                 zLabel: zVal,
-                text: selection.textLabel,
+                text: tx,
                 name: lastPicked.name,
                 color: Fx.castHoverOption(trace, ptNumber, 'bgcolor') || lastPicked.color,
                 borderColor: Fx.castHoverOption(trace, ptNumber, 'bordercolor'),

Diff for: src/traces/cone/attributes.js

@@ -142,6 +142,10 @@ fromMesh3d.forEach(function(k) {
     attrs[k] = mesh3dAttrs[k];
 });
 
-attrs.hoverinfo = extendFlat({}, baseAttrs.hoverinfo, {editType: 'calc'});
+attrs.hoverinfo = extendFlat({}, baseAttrs.hoverinfo, {
+    editType: 'calc',
+    flags: ['x', 'y', 'z', 'u', 'v', 'w', 'norm', 'text', 'name'],
+    dflt: 'x+y+z+norm+text'
+});
 
 module.exports = attrs;

Diff for: src/traces/cone/convert.js

@@ -29,10 +29,20 @@ proto.handlePick = function(selection) {
     if(selection.object === this.pts) {
         var selectIndex = selection.index = selection.data.index;
 
+        var uu = this.data.u[selectIndex];
+        var vv = this.data.v[selectIndex];
+        var ww = this.data.w[selectIndex];
+
         selection.traceCoordinate = [
             this.data.x[selectIndex],
             this.data.y[selectIndex],
-            this.data.z[selectIndex]
+            this.data.z[selectIndex],
+
+            uu,
+            vv,
+            ww,
+
+            Math.sqrt(uu * uu + vv * vv + ww * ww)
         ];
 
         var text = this.data.text;

Diff for: test/image/mocks/gl3d_cone-wind.json

@@ -129167,6 +129167,8 @@
       "colorscale": "Viridis",
       "cmin": 0,
       "cmax": 1
+      "hoverinfo": "u+v+w+text",
+      "text": "-> wind <-"
     }
   ],
   "layout": {