separate hide function

Author: archmoj

src/plots/cartesian/axes.js

@@ -2184,9 +2184,7 @@ axes.drawOne = function(gd, ax, opts) {
             labelFns: axes.makeLabelFns(ax, mainLinePosition)
         };
 
-        if(ax._anchorAxis) {
-            opts.grid = plotinfo.gridlayer.select('.' + ax._anchorAxis._id);
-        }
+        opts.grid = plotinfo.gridlayer.select('.' + ax._id);
 
         return axes.drawLabels(gd, ax, opts);
     });
@@ -3047,9 +3045,8 @@ axes.drawLabels = function(gd, ax, opts) {
         });
     }
 
-    ax._hideOutOfRangeInsideTickLabels = undefined;
-    if((ax.ticklabelposition || '').indexOf('inside') !== -1) {
-        ax._hideOutOfRangeInsideTickLabels = function() {
+    ax._hideOutOfRangeInsideTickLabels = function() {
+        if((ax.ticklabelposition || '').indexOf('inside') !== -1) {
             var rl = Lib.simpleMap(ax.range, ax.r2l);
 
             // hide inside tick labels that go outside axis end points
@@ -3087,24 +3084,30 @@ axes.drawLabels = function(gd, ax, opts) {
                 } // TODO: hide mathjax?
             });
 
-            var anchorAx = ax._anchorAxis || {};
-
-            if((anchorAx.ticklabelposition || '').indexOf('inside') !== -1) {
-                var grid = opts.grid;
-                if(grid) {
-                    grid.each(function() {
-                        d3.select(this).selectAll('path').each(function(d) {
-                            var q = anchorAx.l2p(d.x) + anchorAx._offset;
+            if(ax._anchorAxis) {
+                ax._anchorAxis._visibleLabelMin = visibleLabelMin;
+                ax._anchorAxis._visibleLabelMax = visibleLabelMax;
+            }
+        }
+    };
 
-                            if(q < visibleLabelMax && q > visibleLabelMin) {
-                                d3.select(this).style({ opacity: 0 });
-                            }
-                        });
+    ax._hideCounterAxisInsideTickLabels = function() {
+        var anchorAx = ax._anchorAxis || {};
+        if((anchorAx.ticklabelposition || '').indexOf('inside') !== -1) {
+            var grid = opts.grid;
+            if(grid) {
+                grid.each(function() {
+                    d3.select(this).selectAll('path').each(function(d) {
+                        var q = ax.l2p(d.x) + ax._offset;
+
+                        if(q < ax._visibleLabelMax && q > ax._visibleLabelMin) {
+                            d3.select(this).style({ opacity: 0 });
+                        }
                     });
-                }
+                });
             }
-        };
-    }
+        }
+    };
 
     // make sure all labels are correctly positioned at their base angle
     // the positionLabels call above is only for newly drawn labels.

src/plots/plots.js

@@ -2076,17 +2076,21 @@ plots.doAutoMargin = function(gd) {
         }
     }
 
-    hideOutOfRangeInsideTickLabels(gd);
+    hideInsideTickLabels(gd);
 };
 
-function hideOutOfRangeInsideTickLabels(gd) {
+function hideInsideTickLabels(gd) {
     var axList = axisIDs.list(gd, '', true);
-    for(var i = 0; i < axList.length; i++) {
-        var ax = axList[i];
 
-        var hideFn = ax._hideOutOfRangeInsideTickLabels;
-        if(hideFn) hideFn();
-    }
+    [
+        '_hideOutOfRangeInsideTickLabels',
+        '_hideCounterAxisInsideTickLabels'
+    ].forEach(function(k) {
+        for(var i = 0; i < axList.length; i++) {
+            var hideFn = axList[i][k];
+            if(hideFn) hideFn();
+        }
+    });
 }
 
 var marginKeys = ['l', 'r', 't', 'b', 'p', 'w', 'h'];