Merge pull request #6163 from plotly/blank-ticklabels

improve handling blank tick labels
plotly · Apr 1, 2022 · 646dfd6 · 646dfd6
2 parents 0774d15 + 0734767
commit 646dfd6
Showing 2 changed files with 62 additions and 62 deletions.
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -946,7 +946,7 @@ axes.calcTicks = function calcTicks(ax, opts) {
 
     var lastVisibleHead;
     var hideLabel = function(tick) {
-        tick.text = ' '; // don't use an empty string here which can confuse automargin (issue 5132)
+        tick.text = '';
         ax._prevDateHead = lastVisibleHead;
     };
 
@@ -3008,7 +3008,7 @@ axes.drawLabels = function(gd, ax, opts) {
     var axLetter = axId.charAt(0);
     var cls = opts.cls || axId + 'tick';
 
-    var vals = opts.vals;
+    var vals = opts.vals.filter(function(e) { return e.text; });
 
     var labelFns = opts.labelFns;
     var tickAngle = opts.secondary ? 0 : ax.tickangle;