@spider-analyzer/timeline 5.0.5 → 5.0.6

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 5.0.6
+
+- Fix: don't issue the first `onLoadHisto` with a stale `intervalMs`
+  when the consumer hasn't measured a valid width yet. `getItems` now
+  short-circuits the fetch while `histoWidth <= 0`, and the width-change
+  effect re-runs `getItems` whenever a domain exists (not only when the
+  prior call marked `isActive=true`). Previously the first call fired
+  with `histoWidth=0`, fell back to `domainSpan / 1 tick` and divided by
+  `barsBetweenTicks` — giving a coarse interval like 3 h over a 24 h
+  window; the subsequent width arrival then never retriggered the load
+  because `isActive` was already true and the delta check failed.
+
 ## 5.0.5
 
 - Fix: compute intervalMs defensively in `getItems`. When d3's

package/dist/index.js

@@ -2089,7 +2089,7 @@ var TimeLineInner = react.forwardRef(function TimeLine(props, ref) {
       propsRef.current.smallestResolution.asMilliseconds()
     ]);
     patchState({ histoWidth: histoWidth2, isActive: true, ticks: ticks2 });
-    if (shouldReload && intervalMs) {
+    if (shouldReload && intervalMs && histoWidth2 > 0) {
       widthOfLastUpdateRef.current = p.width;
       patchState({ waitForLoad: true });
       p.onLoadHisto(intervalMs, domain2.min, domain2.max);
@@ -2119,12 +2119,11 @@ var TimeLineInner = react.forwardRef(function TimeLine(props, ref) {
       return;
     }
     const s = stateRef.current;
-    if (s.isActive) {
+    patchState(computeMarginAndWidth(props));
+    if (s.domain) {
       const big = widthOfLastUpdateRef.current !== null && Math.abs((props.width ?? 0) - (widthOfLastUpdateRef.current ?? 0)) > 30;
-      getItems(props, s.domain, big);
-      patchState(computeMarginAndWidth(props));
-    } else {
-      patchState(computeMarginAndWidth(props));
+      const shouldReload = !s.isActive || big || widthOfLastUpdateRef.current === null;
+      getItems(props, s.domain, shouldReload);
     }
     if (stateRef.current.domain) {
       const res = checkAndCorrectDomain({ domain: stateRef.current.domain });