allow choropleth trace to have (trace-wide) string 'text' values

Author: etpinard

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

@@ -34,11 +34,9 @@ module.exports = extendFlat({
         editType: 'calc',
         description: 'Sets the color values.'
     },
-    text: {
-        valType: 'data_array',
-        editType: 'calc',
+    text: extendFlat({}, ScatterGeoAttrs.text, {
         description: 'Sets the text elements associated with each location.'
-    },
+    }),
     marker: {
         line: {
             color: ScatterGeoMarkerLineAttrs.color,

Diff for: src/traces/choropleth/hover.js

@@ -79,7 +79,14 @@ function makeHoverInfo(pointData, trace, pt, axis) {
     }
 
     if(hasZ) text.push(formatter(pt.z));
-    if(hasText) text.push(pt.tx);
+    if(hasText) {
+        var tx;
+
+        if(pt.tx) tx = pt.tx;
+        else if(trace.text) tx = trace.text;
+
+        if(!Array.isArray(tx)) text.push(tx);
+    }
 
     pointData.extraText = text.join('<br>');
 }

Diff for: test/jasmine/tests/choropleth_test.js

@@ -93,6 +93,19 @@ describe('Test choropleth hover:', function() {
         .then(done);
     });
 
+    it('should generate hover label info (\'text\' single value case)', function(done) {
+        var fig = Lib.extendDeep({}, require('@mocks/geo_first.json'));
+        fig.data[1].text = 'tExT';
+        fig.data[1].hoverinfo = 'text';
+
+        run(
+            [400, 160],
+            fig,
+            ['tExT', null]
+        )
+        .then(done);
+    });
+
     it('should generate hover label info (\'text\' array case)', function(done) {
         var fig = Lib.extendDeep({}, require('@mocks/geo_first.json'));
         fig.data[1].text = ['tExT', 'TeXt', '-text-'];