Skip to content

Commit 8d010c0

Browse files
committed
drop deprecated autotick from cartesian axes
1 parent 2be6fcc commit 8d010c0

File tree

4 files changed

+4
-25
lines changed

4 files changed

+4
-25
lines changed

src/plot_api/helpers.js

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -81,15 +81,6 @@ exports.cleanLayout = function(layout) {
8181

8282
// prune empty domain arrays made before the new nestedProperty
8383
if(emptyContainer(ax, 'domain')) delete ax.domain;
84-
85-
// autotick -> tickmode
86-
if(ax.autotick !== undefined) {
87-
if(ax.tickmode === undefined) {
88-
ax.tickmode = ax.autotick ? 'auto' : 'linear';
89-
}
90-
delete ax.autotick;
91-
}
92-
9384
}
9485
}
9586

src/plots/cartesian/layout_attributes.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1230,16 +1230,4 @@ module.exports = {
12301230
].join(' ')
12311231
},
12321232
editType: 'calc',
1233-
1234-
_deprecated: {
1235-
autotick: {
1236-
valType: 'boolean',
1237-
editType: 'ticks',
1238-
description: [
1239-
'Obsolete.',
1240-
'Set `tickmode` to *auto* for old `autotick` *true* behavior.',
1241-
'Set `tickmode` to *linear* for `autotick` *false*.'
1242-
].join(' ')
1243-
},
1244-
}
12451233
};

src/plots/cartesian/set_convert.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ function isValidCategory(v) {
5858
* Creates/updates these conversion functions, and a few more utilities
5959
* like cleanRange, and makeCalcdata
6060
*
61-
* also clears the autotick constraints ._minDtick, ._forceTick0
61+
* also clears ._minDtick, ._forceTick0
6262
*/
6363
module.exports = function setConvert(ax, fullLayout) {
6464
fullLayout = fullLayout || {};

src/plots/gl3d/layout/tick_marks.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,9 @@ function computeTickMarks(scene) {
5252
axes.range[1] += 1;
5353
}
5454
// this is necessary to short-circuit the 'y' handling
55-
// in autotick part of calcTicks... Treating all axes as 'y' in this case
56-
// running the autoticks here, then setting
57-
// autoticks to false to get around the 2D handling in calcTicks.
55+
// in tickmode part of calcTicks... Treating all axes as 'y' in this case
56+
// running the tickmode here, then setting
57+
// automode to linear to get around the 2D handling in calcTicks.
5858
var tickModeCached = axes.tickmode;
5959
if(axes.tickmode === 'auto') {
6060
axes.tickmode = 'linear';

0 commit comments

Comments
 (0)