no need to labelStandoff when showticklabels is false

etpinard · etpinard · commit d951f6a8fe4d · 2017-07-25T10:26:52.000-04:00
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -1705,10 +1705,8 @@ axes.doTicks = function(gd, axid, skipTitle) {
         labelShift = ax.ticklen * Math.sin(caRad);
     }
 
-    if(ax.ticks === 'outside' || ax.showline) {
-        var tickFontSize = ax.tickfont ? ax.tickfont.size : 12;
-
-        labelStandoff += 0.2 * tickFontSize;
+    if(ax.showticklabels && (ax.ticks === 'outside' || ax.showline)) {
+        labelStandoff += 0.2 * ax.tickfont.size;
     }
 
     // positioning arguments for x vs y axes

Original file line number	Diff line number	Diff line change
`@@ -1705,10 +1705,8 @@ axes.doTicks = function(gd, axid, skipTitle) {`
`1705`	`1705`	`labelShift = ax.ticklen * Math.sin(caRad);`
`1706`	`1706`	`}`
`1707`	`1707`
`1708`		`- if(ax.ticks === 'outside' \|\| ax.showline) {`
`1709`		`- var tickFontSize = ax.tickfont ? ax.tickfont.size : 12;`
`1710`		`-`
`1711`		`- labelStandoff += 0.2 * tickFontSize;`
	`1708`	`+ if(ax.showticklabels && (ax.ticks === 'outside' \|\| ax.showline)) {`
	`1709`	`+ labelStandoff += 0.2 * ax.tickfont.size;`
`1712`	`1710`	`}`
`1713`	`1711`
`1714`	`1712`	`// positioning arguments for x vs y axes`