fix issue 4902

archmoj · archmoj · commit 904351addf82 · 2020-06-05T10:40:03.000-04:00
diff --git a/src/lib/index.js b/src/lib/index.js
@@ -257,10 +257,10 @@ lib.bBoxIntersect = function(a, b, pad) {
  * func: the function to apply
  * x1, x2: optional extra args
  */
-lib.simpleMap = function(array, func, x1, x2) {
+lib.simpleMap = function(array, func, x1, x2, opts) {
     var len = array.length;
     var out = new Array(len);
-    for(var i = 0; i < len; i++) out[i] = func(array[i], x1, x2);
+    for(var i = 0; i < len; i++) out[i] = func(array[i], x1, x2, opts);
     return out;
 };
 
diff --git a/src/plots/cartesian/axes.js b/src/plots/cartesian/axes.js
@@ -509,8 +509,8 @@ function autoShiftMonthBins(binStart, data, dtick, dataMin, calendar) {
 // ----------------------------------------------------
 
 // ensure we have tick0, dtick, and tick rounding calculated
-axes.prepTicks = function(ax) {
-    var rng = Lib.simpleMap(ax.range, ax.r2l);
+axes.prepTicks = function(ax, opts) {
+    var rng = Lib.simpleMap(ax.range, ax.r2l, undefined, undefined, opts);
 
     // calculate max number of (auto) ticks to display based on plot size
     if(ax.tickmode === 'auto' || !ax.dtick) {
@@ -563,16 +563,16 @@ axes.prepTicks = function(ax) {
 // if ticks are set to automatic, determine the right values (tick0,dtick)
 // in any case, set tickround to # of digits to round tick labels to,
 // or codes to this effect for log and date scales
-axes.calcTicks = function calcTicks(ax) {
-    axes.prepTicks(ax);
-    var rng = Lib.simpleMap(ax.range, ax.r2l);
+axes.calcTicks = function calcTicks(ax, opts) {
+    axes.prepTicks(ax, opts);
+    var rng = Lib.simpleMap(ax.range, ax.r2l, undefined, undefined, opts);
 
     // now that we've figured out the auto values for formatting
     // in case we're missing some ticktext, we can break out for array ticks
     if(ax.tickmode === 'array') return arrayTicks(ax);
 
     // find the first tick
-    ax._tmin = axes.tickFirst(ax);
+    ax._tmin = axes.tickFirst(ax, opts);
 
     // add a tiny bit so we get ticks which may have rounded out
     var exRng = expandRange(rng);
@@ -962,9 +962,9 @@ axes.tickIncrement = function(x, dtick, axrev, calendar) {
 };
 
 // calculate the first tick on an axis
-axes.tickFirst = function(ax) {
+axes.tickFirst = function(ax, opts) {
     var r2l = ax.r2l || Number;
-    var rng = Lib.simpleMap(ax.range, r2l);
+    var rng = Lib.simpleMap(ax.range, r2l, undefined, undefined, opts);
     var axrev = rng[1] < rng[0];
     var sRound = axrev ? Math.floor : Math.ceil;
     // add a tiny extra bit to make sure we get ticks
diff --git a/src/plots/gl3d/layout/tick_marks.js b/src/plots/gl3d/layout/tick_marks.js
@@ -69,7 +69,7 @@ function computeTickMarks(scene) {
                 var nticks = axes.nticks || Lib.constrain((axes._length / 40), 4, 9);
                 Axes.autoTicks(axes, Math.abs(axes.range[1] - axes.range[0]) / nticks);
             }
-            var dataTicks = Axes.calcTicks(axes);
+            var dataTicks = Axes.calcTicks(axes, { msUTC: true });
             for(var j = 0; j < dataTicks.length; ++j) {
                 dataTicks[j].x = dataTicks[j].x * scene.dataScale[i];
 
diff --git a/test/image/baselines/gl3d_ticks-milliseconds.png b/test/image/baselines/gl3d_ticks-milliseconds.png
diff --git a/test/image/mocks/gl3d_ticks-milliseconds.json b/test/image/mocks/gl3d_ticks-milliseconds.json
@@ -0,0 +1,26 @@
+{
+    "data": [{
+        "type": "scatter3d",
+        "x": ["1970-01-01 01:00:00", "1970-01-01 13:00:00"],
+        "y": ["1970-01-01 01:00:00", "1970-01-01 02:00:00"],
+        "z": ["1970-01-01 01:00:00", "1970-01-01 01:30:00"]
+    }],
+    "layout": {
+        "width": 400,
+        "height": 400,
+        "margin": {
+            "t": 10,
+            "b": 10,
+            "l": 10,
+            "r": 10
+        },
+        "scene": {
+            "xaxis": { "nticks": 12, "type": "date" },
+            "yaxis": { "nticks": 12, "type": "date" },
+            "zaxis": { "nticks": 12, "type": "date" },
+            "camera": {
+                "eye": { "x": 0.25, "y": 2.5, "z": 0.25 }
+            }
+        }
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ function computeTickMarks(scene) {`
`69`	`69`	`var nticks = axes.nticks \|\| Lib.constrain((axes._length / 40), 4, 9);`
`70`	`70`	`Axes.autoTicks(axes, Math.abs(axes.range[1] - axes.range[0]) / nticks);`
`71`	`71`	`}`
`72`		`- var dataTicks = Axes.calcTicks(axes);`
	`72`	`+ var dataTicks = Axes.calcTicks(axes, { msUTC: true });`
`73`	`73`	`for(var j = 0; j < dataTicks.length; ++j) {`
`74`	`74`	`dataTicks[j].x = dataTicks[j].x * scene.dataScale[i];`
`75`	`75`