Revert "Fix for #7416: Hidden ticklabels with ticklabelposition "inside" take up plot space "

Author: archmoj

draftlogs/7417_fix.md

@@ -2972,13 +2972,11 @@ function calcLabelLevelBbox(ax, cls, mainLinePositionShift) {
             // (like in fixLabelOverlaps) instead and use Axes.getPxPosition
             // together with the makeLabelFns outputs and `tickangle`
             // to compute one bbox per (tick value x tick style)
-            if (thisLabel.node().style.display !== 'none') {
-                var bb = Drawing.bBox(thisLabel.node().parentNode);
-                top = Math.min(top, bb.top);
-                bottom = Math.max(bottom, bb.bottom);
-                left = Math.min(left, bb.left);
-                right = Math.max(right, bb.right);
-            }
+            var bb = Drawing.bBox(thisLabel.node().parentNode);
+            top = Math.min(top, bb.top);
+            bottom = Math.max(bottom, bb.bottom);
+            left = Math.min(left, bb.left);
+            right = Math.max(right, bb.right);
         });
     } else {
         var dummyCalc = axes.makeLabelFns(ax, mainLinePositionShift);
@@ -3671,7 +3669,7 @@ axes.drawLabels = function(gd, ax, opts) {
                     'text-anchor': anchor
                 });
 
-                thisText.style('display', null); // visible
+                thisText.style('opacity', 1); // visible
 
                 if(ax._adjustTickLabelsOverflow) {
                     ax._adjustTickLabelsOverflow();
@@ -3729,9 +3727,9 @@ axes.drawLabels = function(gd, ax, opts) {
 
                 var t = thisLabel.select('text');
                 if(adjust) {
-                    if(hideOverflow) t.style('display', 'none'); // hidden
-                } else if(t.node().style.display !== 'none'){
-                    t.style('display', null);
+                    if(hideOverflow) t.style('opacity', 0); // hidden
+                } else {
+                    t.style('opacity', 1); // visible
 
                     if(side === 'bottom' || side === 'right') {
                         visibleLabelMin = Math.min(visibleLabelMin, isX ? bb.top : bb.left);
@@ -3808,7 +3806,7 @@ axes.drawLabels = function(gd, ax, opts) {
                                 q > ax['_visibleLabelMin_' + anchorAx._id]
                             ) {
                                 t.style('display', 'none'); // hidden
-                            } else if(e.K === 'tick' && !idx && t.node().style.display !== 'none') {
+                            } else if(e.K === 'tick' && !idx) {
                                 t.style('display', null); // visible
                             }
                         });