Merge pull request #2698 from plotly/scatter3d-scalar-hovertext-fix

Fix scatter3d scalar hovertext handling

Author: etpinard

src/traces/scatter3d/convert.js

@@ -50,28 +50,33 @@ proto.handlePick = function(selection) {
         (selection.object === this.linePlot ||
          selection.object === this.delaunayMesh ||
          selection.object === this.textMarkers ||
-         selection.object === this.scatterPlot)) {
+         selection.object === this.scatterPlot)
+    ) {
+        var ind = selection.index = selection.data.index;
+
         if(selection.object.highlight) {
             selection.object.highlight(null);
         }
         if(this.scatterPlot) {
             selection.object = this.scatterPlot;
             this.scatterPlot.highlight(selection.data);
         }
+
+        selection.textLabel = '';
         if(this.textLabels) {
-            if(this.textLabels[selection.data.index] !== undefined) {
-                selection.textLabel = this.textLabels[selection.data.index];
+            if(Array.isArray(this.textLabels)) {
+                if(this.textLabels[ind] || this.textLabels[ind] === 0) {
+                    selection.textLabel = this.textLabels[ind];
+                }
             } else {
                 selection.textLabel = this.textLabels;
             }
         }
-        else selection.textLabel = '';
 
-        var selectIndex = selection.index = selection.data.index;
         selection.traceCoordinate = [
-            this.data.x[selectIndex],
-            this.data.y[selectIndex],
-            this.data.z[selectIndex]
+            this.data.x[ind],
+            this.data.y[ind],
+            this.data.z[ind]
         ];
 
         return true;

test/jasmine/tests/gl3d_plot_interact_test.js

@@ -203,6 +203,11 @@ describe('@gl Test gl3d plots', function() {
         .then(function() {
             assertHoverText(null, null, null, 'Clementine');
 
+            return Plotly.restyle(gd, 'hovertext', 'HEY');
+        })
+        .then(function() {
+            assertHoverText(null, null, null, 'HEY');
+
             return Plotly.restyle(gd, 'hoverinfo', 'z');
         })
         .then(function() {