fix scattermapbox handling of '' text items

etpinard · etpinard · commit fd9cb3ff0c4e · 2017-01-04T17:31:08.000-05:00
diff --git a/src/traces/scattermapbox/hover.js b/src/traces/scattermapbox/hover.js
@@ -86,7 +86,8 @@ function getExtraText(trace, di) {
     else if(hasLat) text.push('lat: ' + format(lonlat[1]));
 
     if(isAll || hoverinfo.indexOf('text') !== -1) {
-        text.push(di.tx || trace.text);
+        var tx = di.tx || trace.text;
+        if(!Array.isArray(tx)) text.push(tx);
     }
 
     return text.join('<br>');
diff --git a/test/jasmine/tests/scattermapbox_test.js b/test/jasmine/tests/scattermapbox_test.js
@@ -494,6 +494,36 @@ describe('scattermapbox hover', function() {
         expect(out.color).toEqual('#1f77b4');
     });
 
+    it('should skip over blank and non-string text items', function(done) {
+        var xval = 11,
+            yval = 11,
+            out;
+
+        Plotly.restyle(gd, 'text', [['', 'B', 'C']]).then(function() {
+            out = hoverPoints(getPointData(gd), xval, yval)[0];
+            expect(out.extraText).toEqual('(10°, 10°)');
+
+            return Plotly.restyle(gd, 'text', [[null, 'B', 'C']]);
+        })
+        .then(function() {
+            out = hoverPoints(getPointData(gd), xval, yval)[0];
+            expect(out.extraText).toEqual('(10°, 10°)');
+
+            return Plotly.restyle(gd, 'text', [[false, 'B', 'C']]);
+        })
+        .then(function() {
+            out = hoverPoints(getPointData(gd), xval, yval)[0];
+            expect(out.extraText).toEqual('(10°, 10°)');
+
+            return Plotly.restyle(gd, 'text', [['A', 'B', 'C']]);
+        })
+        .then(function() {
+            out = hoverPoints(getPointData(gd), xval, yval)[0];
+            expect(out.extraText).toEqual('(10°, 10°)<br>A');
+        })
+        .then(done);
+    });
+
     it('should generate hover label info (positive winding case)', function() {
         var xval = 11 + 720,
             yval = 11;

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,8 @@ function getExtraText(trace, di) {`
`86`	`86`	`else if(hasLat) text.push('lat: ' + format(lonlat[1]));`
`87`	`87`
`88`	`88`	`if(isAll \|\| hoverinfo.indexOf('text') !== -1) {`
`89`		`- text.push(di.tx \|\| trace.text);`
	`89`	`+ var tx = di.tx \|\| trace.text;`
	`90`	`+ if(!Array.isArray(tx)) text.push(tx);`
`90`	`91`	`}`
`91`	`92`
`92`	`93`	`return text.join('<br>');`