add extra axis padding when bars have outside text labels

Author: emilykl

src/traces/bar/cross_trace_calc.js

@@ -9,6 +9,8 @@ var Axes = require('../../plots/cartesian/axes');
 var getAxisGroup = require('../../plots/cartesian/constraints').getAxisGroup;
 var Sieve = require('./sieve.js');
 
+var TEXTPAD = require('./constants').TEXTPAD;
+
 /*
  * Bar chart stacking/grouping positioning and autoscaling calculations
  * for each direction separately calculate the ranges and positions
@@ -565,13 +567,32 @@ function setBaseAndTop(sa, sieve) {
             }
         }
 
+        const textPadding = estimateExtraPaddingForText(fullTrace);
         fullTrace._extremes[sa._id] = Axes.findExtremes(sa, pts, {
             tozero: tozero,
-            padded: true
+            padded: true,
+            ppad: textPadding
         });
     }
 }
 
+// Very lightweight estimate of extra padding needed
+// to accommodate outside text labels on bars
+// Only considers vertical bars with 'outside' textposition for now
+// This solves the most common scenario of a simple vertical bar chart
+// with textposition set to 'outside'
+// More complex scenarios (horizontal bars, multi-line text labels)
+// are not (yet) handled here
+function estimateExtraPaddingForText(trace) {
+    if (trace.orientation === 'v' && (trace.text || trace.texttemplate) && trace.textposition == 'outside') {
+        // could count <br> elements here
+        // but before that, need to make sure we are only
+        // adding padding on the side(s) where it is needed
+        return trace.outsidetextfont.size + TEXTPAD;
+    }
+    return 0;
+}
+
 function stackBars(sa, sieve, opts) {
     var sLetter = getAxisLetter(sa);
     var calcTraces = sieve.traces;
@@ -637,13 +658,16 @@ function stackBars(sa, sieve, opts) {
             }
         }
 
+        const textPadding = estimateExtraPaddingForText(fullTrace);
+
         // if barnorm is set, let normalizeBars update the axis range
         if (!opts.norm) {
             fullTrace._extremes[sa._id] = Axes.findExtremes(sa, pts, {
                 // N.B. we don't stack base with 'base',
                 // so set tozero:true always!
                 tozero: true,
-                padded: true
+                padded: true,
+                ppad: textPadding
             });
         }
     }