Update src/traces/bar/cross_trace_calc.js

Co-authored-by: Cameron DeCoster <cameron.decoster@gmail.com>

Author: emilykl

src/traces/bar/cross_trace_calc.js

@@ -1,17 +1,16 @@
 'use strict';
 
-var isNumeric = require('fast-isnumeric');
-var isArrayOrTypedArray = require('../../lib').isArrayOrTypedArray;
-var BADNUM = require('../../constants/numerical').BADNUM;
+const isNumeric = require('fast-isnumeric');
+const { isArrayOrTypedArray } = require('../../lib');
+const { BADNUM } = require('../../constants/numerical');
 
-var Registry = require('../../registry');
-var Axes = require('../../plots/cartesian/axes');
-var getAxisGroup = require('../../plots/cartesian/constraints').getAxisGroup;
-var Sieve = require('./sieve.js');
+const Registry = require('../../registry');
+const Axes = require('../../plots/cartesian/axes');
+const { getAxisGroup } = require('../../plots/cartesian/constraints');
+const Sieve = require('./sieve.js');
 
-var TEXTPAD = require('./constants').TEXTPAD;
-var LINE_SPACING = require('../../constants/alignment').LINE_SPACING;
-var BR_TAG_ALL = require('../../lib/svg_text_utils').BR_TAG_ALL;
+const { TEXTPAD, LINE_SPACING } = require('./constants');
+const { BR_TAG_ALL } = require('../../lib/svg_text_utils');
 
 /*
  * Bar chart stacking/grouping positioning and autoscaling calculations
@@ -569,12 +568,12 @@ function setBaseAndTop(sa, sieve) {
             }
         }
 
-        const extraPad = estimateAxisPaddingForText(fullTrace, calcTrace);
+        const { ppadminus, ppadplus } = estimateAxisPaddingForText(fullTrace, calcTrace);
         fullTrace._extremes[sa._id] = Axes.findExtremes(sa, pts, {
-            tozero: tozero,
+            tozero,
             padded: true,
-            ppadplus: extraPad.ppadplus,
-            ppadminus: extraPad.ppadminus
+            ppadplus,
+            ppadminus
         });
     }
 }
@@ -646,14 +645,14 @@ function stackBars(sa, sieve, opts) {
 
         // if barnorm is set, let normalizeBars update the axis range
         if (!opts.norm) {
-            const extraPad = estimateAxisPaddingForText(fullTrace, calcTrace);
+            const { ppadminus, ppadplus } = estimateAxisPaddingForText(fullTrace, calcTrace);
             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,
-                ppadplus: extraPad.ppadplus,
-                ppadminus: extraPad.ppadminus
+                ppadplus,
+                ppadminus
             });
         }
     }
@@ -757,12 +756,12 @@ function normalizeBars(sa, sieve, opts) {
             }
         }
 
-        const extraPad = estimateAxisPaddingForText(fullTrace, calcTrace);
+        const { ppadminus, ppadplus } = estimateAxisPaddingForText(fullTrace, calcTrace);
         fullTrace._extremes[sa._id] = Axes.findExtremes(sa, pts, {
-            tozero: tozero,
-            padded: padded,
-            ppadplus: extraPad.ppadplus,
-            ppadminus: extraPad.ppadminus
+            tozero,
+            padded,
+            ppadplus,
+            ppadminus
         });
     }
 }
@@ -784,15 +783,15 @@ function estimateAxisPaddingForText(trace, calcTrace) {
     if (
         trace.orientation === 'v' &&
         (trace.text || trace.texttemplate) &&
-        trace.textposition == 'outside' &&
-        (trace.textangle == 'auto' || trace.textangle == 0)
+        trace.textposition === 'outside' &&
+        (trace.textangle === 'auto' || trace.textangle === 0)
     ) {
         // count number of lines by counting <br> elements
         function countLines(text) {
             if (!text || typeof text !== 'string') return 0;
             return (text.match(BR_TAG_ALL) || []).length + 1;
         }
-        var nLines = trace.texttemplate
+        const nLines = trace.texttemplate
             ? countLines(trace.texttemplate)
             : isArrayOrTypedArray(trace.text)
               ? Math.max(...trace.text.map((t) => countLines(t)))