heatmaps: part 8

website/src/heatmap/grid.js

@@ -113,7 +113,7 @@ export function createAverageGrid({
           if (col < 0 || col >= cols || dates.length === 0) {
             return emptyRange();
           }
-          const start = Math.floor((col * dates.length) / cols);
+          const start = Math.ceil((col * dates.length) / cols);
           const end = Math.floor(((col + 1) * dates.length - 1) / cols);
           return { start, end: clamp(end, start, dates.length - 1) };
         },

website/src/heatmap/index.js

@@ -109,7 +109,10 @@ function loadRange() {
   let failed = 0;
   updateStatus(completed, currentDates.length, failed);
 
-  if (!missing.length) return;
+  if (!missing.length) {
+    abortController = undefined;
+    return;
+  }
 
   let cursor = 0;
   const workers = Array.from({
@@ -264,7 +267,7 @@ function createRangeControls() {
     onChange(choice) {
       fromChoice = choice;
       if (fromChoice.date > toChoice.date) {
-        toChoice = findMatchingChoice(toChoices, fromChoice);
+        toChoice = findSameLabelChoice(toChoices, fromChoice, toChoices[0]);
         toSelect.set(toChoice);
       }
       setRange(fromChoice.date, toChoice.date);
@@ -280,7 +283,7 @@ function createRangeControls() {
     onChange(choice) {
       toChoice = choice;
       if (fromChoice.date > toChoice.date) {
-        fromChoice = findMatchingChoice(fromChoices, toChoice);
+        fromChoice = findSameLabelChoice(fromChoices, toChoice, fromChoices[0]);
         fromSelect.set(fromChoice);
       }
       setRange(fromChoice.date, toChoice.date);
@@ -330,10 +333,11 @@ function rangeChoiceLabel(choice) {
 
 /**
  * @param {readonly RangeChoice[]} choices
- * @param {RangeChoice} selected
+ * @param {RangeChoice} choice
+ * @param {RangeChoice} fallback
  */
-function findMatchingChoice(choices, selected) {
-  return choices.find((choice) => choice.label === selected.label) ?? choices[0];
+function findSameLabelChoice(choices, choice, fallback) {
+  return choices.find((candidate) => candidate.label === choice.label) ?? fallback;
 }
 
 /**