Merge pull request #6069 from plotly/histogram-x-y-texttemplate

Fix x/y `texttemplate` for `histogram`, `bar`, `funnel` and `waterfall` traces

Author: archmoj

src/traces/bar/plot.js

@@ -631,10 +631,11 @@ function calcTexttemplate(fullLayout, cd, index, xa, ya) {
     if(!texttemplate) return '';
     var isWaterfall = (trace.type === 'waterfall');
     var isFunnel = (trace.type === 'funnel');
+    var isHorizontal = trace.orientation === 'h';
 
     var pLetter, pAxis;
     var vLetter, vAxis;
-    if(trace.orientation === 'h') {
+    if(isHorizontal) {
         pLetter = 'y';
         pAxis = ya;
         vLetter = 'x';
@@ -669,6 +670,11 @@ function calcTexttemplate(fullLayout, cd, index, xa, ya) {
     var pt = {};
     appendArrayPointValue(pt, trace, cdi.i);
 
+    if(pt.x === undefined) pt.x = isHorizontal ? obj.value : obj.label;
+    if(pt.y === undefined) pt.y = isHorizontal ? obj.label : obj.value;
+    if(pt.xLabel === undefined) pt.xLabel = isHorizontal ? obj.valueLabel : obj.labelLabel;
+    if(pt.yLabel === undefined) pt.yLabel = isHorizontal ? obj.labelLabel : obj.valueLabel;
+
     if(isWaterfall) {
         obj.delta = +cdi.rawS || cdi.s;
         obj.deltaLabel = formatNumber(obj.delta);

src/traces/histogram/calc.js

@@ -15,8 +15,9 @@ var getBinSpanLabelRound = require('./bin_label_vals');
 function calc(gd, trace) {
     var pos = [];
     var size = [];
-    var pa = Axes.getFromId(gd, trace.orientation === 'h' ? trace.yaxis : trace.xaxis);
-    var mainData = trace.orientation === 'h' ? 'y' : 'x';
+    var isHorizontal = trace.orientation === 'h';
+    var pa = Axes.getFromId(gd, isHorizontal ? trace.yaxis : trace.xaxis);
+    var mainData = isHorizontal ? 'y' : 'x';
     var counterData = {x: 'y', y: 'x'}[mainData];
     var calendar = trace[mainData + 'calendar'];
     var cumulativeSpec = trace.cumulative;

test/image/baselines/bar-marker-line-colorscales.png

@@ -1,6 +1,7 @@
 {
     "data": [
         {
+            "texttemplate": "%{x}",
             "marker": {
                 "color": [
                     0,

test/image/baselines/bar_marker_array.png

@@ -1,6 +1,8 @@
 {
   "data": [
     {
+      "texttemplate": "%{y:(}",
+      "textfont": { "size": 32 },
       "x": [
         1,
         2,

test/image/baselines/date_histogram.png

@@ -1,6 +1,7 @@
 {
     "data": [
         {
+            "texttemplate": "<b>%{y:.1f}</b><br><br>%{x}",
             "x": [
                 "2012-01-01 00:00:00",
                 "2012-02-01 00:00:00",

test/image/mocks/bar-marker-line-colorscales.json

@@ -13,7 +13,7 @@
                 "cmax": 5
             },
             "x": [ 0, 1, 1, 2, 2, 2, 3, 3, 4 ],
-            "texttemplate": "%{value}",
+            "texttemplate": "%{y}",
             "textposition": "inside",
             "textfont": {
                 "size": 64