replace ' = ' for ': ' in scattercarpet hover label prefixes

Author: etpinard

src/traces/scattercarpet/hover.js

@@ -53,7 +53,15 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     var text = [];
 
     function textPart(ax, val) {
-        text.push(((ax.labelprefix && ax.labelprefix.length > 0) ? ax.labelprefix : (ax._hovertitle + ': ')) + val.toFixed(3) + ax.labelsuffix);
+        var prefix;
+
+        if(ax.labelprefix && ax.labelprefix.length > 0) {
+            prefix = ax.labelprefix.replace(/ = $/, '');
+        } else {
+            prefix = ax._hovertitle;
+        }
+
+        text.push(prefix + ': ' + val.toFixed(3) + ax.labelsuffix);
     }
 
     if(parts.indexOf('all') !== -1) parts = ['a', 'b'];
@@ -68,7 +76,7 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     var tj = ij[1] - j0;
 
     var xy = carpet.evalxy([], i0, j0, ti, tj);
-    text.push('y = ' + xy[1].toFixed(3));
+    text.push('y: ' + xy[1].toFixed(3));
 
     newPointData.extraText = text.join('<br>');
 

test/jasmine/tests/carpet_test.js

@@ -591,7 +591,7 @@ describe('scattercarpet hover labels', function() {
 
         run(
             [200, 200], fig,
-            [['a = 0.200', 'b = 3.500', 'y = 2.900'], 'a = 0.2']
+            [['a: 0.200', 'b: 3.500', 'y: 2.900'], 'a = 0.2']
         )
         .then(done);
     });
@@ -602,7 +602,7 @@ describe('scattercarpet hover labels', function() {
 
         run(
             [200, 200], fig,
-            [['a = 0.200', 'y = 2.900'], null]
+            [['a: 0.200', 'y: 2.900'], null]
         )
         .then(done);
     });
@@ -613,7 +613,7 @@ describe('scattercarpet hover labels', function() {
 
         run(
             [200, 200], fig,
-            [['b = 3.500', 'y = 2.900'], null]
+            [['b: 3.500', 'y: 2.900'], null]
         )
         .then(done);
     });