use interval.ceil, if available

observablehq · Aug 4, 2023 · 2afe52a · 2afe52a
1 parent 132ab4f
commit 2afe52a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 12 deletions.
diff --git a/src/marks/axis.js b/src/marks/axis.js
@@ -542,12 +542,13 @@ function axisMark(mark, k, ariaLabel, data, options, initialize) {
         // ordinal time
         // TODO determine whether the interval is utc or local time?
         // TODO assert that the interval is one of utcTick’s known intervals?
+        // TODO add ceil to the RangeIntervalImplementation interface?
         if (isTimeInterval(scale.interval)) {
-          const type = "utc"; // TODO detect whether interval is utc or local
+          const type = "utc";
           const [start, stop] = extent(data);
           if (interval) {
             data = maybeRangeInterval(interval, type).range(start, stop);
-            data = data.map(scale.interval.floor, scale.interval);
+            data = data.map(scale.interval.ceil ?? scale.interval.floor, scale.interval);
           } else {
             if (ticks === undefined) {
               const [min, max] = extent(scale.range());

diff --git a/test/output/timeAxisOrdinalIrregular.svg b/test/output/timeAxisOrdinalIrregular.svg