Fix ticks for small log domains.

d3 · Nov 23, 2019 · a8ef733 · a8ef733
1 parent cf2c15b
commit a8ef733
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/log.js b/src/log.js
@@ -102,7 +102,7 @@ export function loggish(transform) {
           z.push(t);
         }
       }
-      if (!z.length) z = ticks(u, v, n);
+      if (z.length * 2 < n) z = ticks(u, v, n);
     } else {
       z = ticks(i, j, Math.min(j - i, n)).map(pows);
     }

diff --git a/test/log-test.js b/test/log-test.js
@@ -296,6 +296,11 @@ tape("log.ticks() generates the expected power-of-ten ticks for small domains",
   test.deepEqual(s.domain([5, 1]).ticks(), [5, 4, 3, 2, 1]);
   test.deepEqual(s.domain([-1, -5]).ticks(), [-1, -2, -3, -4, -5]);
   test.deepEqual(s.domain([-5, -1]).ticks(), [-5, -4, -3, -2, -1]);
+  test.deepEqual(s.domain([286.9252014, 329.4978332]).ticks(1), [300]);
+  test.deepEqual(s.domain([286.9252014, 329.4978332]).ticks(2), [300]);
+  test.deepEqual(s.domain([286.9252014, 329.4978332]).ticks(3), [300, 320]);
+  test.deepEqual(s.domain([286.9252014, 329.4978332]).ticks(4), [290, 300, 310, 320]);
+  test.deepEqual(s.domain([286.9252014, 329.4978332]).ticks(), [290, 295, 300, 305, 310, 315, 320, 325]);
   test.end();
 });