virtua 0.53.4 → 0.53.5

package/lib/svelte/GridRow.svelte

@@ -26,8 +26,10 @@
 
 <div role="row" aria-rowindex={state.$row + 1} {style}>
   {#each state.$cells as cell (cell.$col)}
-    <!-- Guard for experimental.async: true, which runs the each block before $effect.pre -->
+    <!-- The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update -->
+    <!-- https://svelte.dev/docs/svelte/await-expressions#Breaking-changes -->
     <!-- https://github.com/inokawa/virtua/pull/847 -->
+    <!-- https://github.com/inokawa/virtua/pull/1018 -->
     {#if cell.$col < getAxisLength(cols)}
       <GridCell
         state={cell}

package/lib/svelte/GridRowGroup.svelte

@@ -27,8 +27,10 @@
 <!-- https://www.w3.org/TR/wai-aria-1.2/#rowgroup -->
 <div role="rowgroup" {style}>
   {#each state.$rows as row (row.$row)}
-    <!-- Guard for experimental.async: true, which runs the each block before $effect.pre -->
+    <!-- The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update -->
+    <!-- https://svelte.dev/docs/svelte/await-expressions#Breaking-changes -->
     <!-- https://github.com/inokawa/virtua/pull/847 -->
+    <!-- https://github.com/inokawa/virtua/pull/1018 -->
     {#if row.$row < getAxisLength(rows)}
       <GridRow
         state={row}

package/lib/svelte/VGrid.svelte

@@ -242,8 +242,10 @@
           resizer={driver.$observeItem}
         />
       {:else}
-        <!-- Guard for experimental.async: true, which runs the each block before $effect.pre -->
+        <!-- The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update -->
+        <!-- https://svelte.dev/docs/svelte/await-expressions#Breaking-changes -->
         <!-- https://github.com/inokawa/virtua/pull/847 -->
+        <!-- https://github.com/inokawa/virtua/pull/1018 -->
         {#if state.$row < getAxisLength(rows)}
           <GridRow
             {state}

package/lib/svelte/VMasonry.svelte

@@ -16,6 +16,7 @@
     scrollTo as _scrollTo,
     scrollBy as _scrollBy,
     scrollToIndex as _scrollToIndex,
+    min,
     sort,
   } from "../core/index.js";
   import { defaultGetKey, isSameRange, styleToString } from "./utils.js";
@@ -86,7 +87,11 @@
   let crossSize = $derived(toCrossValue(1 / lanes, gap / lanes - gap));
 
   let indexes = $derived.by(() => {
-    const len = data.length;
+    // The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update
+    // https://svelte.dev/docs/svelte/await-expressions#Breaking-changes
+    // https://github.com/inokawa/virtua/pull/847
+    // https://github.com/inokawa/virtua/pull/1018
+    const len = stateVersion && min(data.length, layout.$getLength());
     const [start, end] = range;
     const arr: number[] = [];
     if (keepMounted) {
@@ -94,17 +99,13 @@
       for (let i = start; i <= end; i++) {
         mounted.add(i);
       }
-      // The store may still have the previous length in async mode, as the range does
-      const storeLen = stateVersion && layout.$getLength();
       for (const index of sort([...mounted])) {
-        if (index < len && index < storeLen) {
+        if (index < len) {
           arr.push(index);
         }
       }
     } else {
       for (let i = start; i <= end; i++) {
-        // Guard for experimental.async: true, which runs the each block before $effect.pre
-        // https://github.com/inokawa/virtua/pull/847
         if (i < len) {
           arr.push(i);
         }

package/lib/svelte/Virtualizer.svelte

@@ -16,6 +16,7 @@
     createVirtualStore,
     createListLayout,
     getScrollSize as _getScrollSize,
+    min,
     sort,
   } from "../core/index.js";
   import { defaultGetKey, isSameRange, styleToString } from "./utils.js";
@@ -83,7 +84,11 @@
   let totalSize = $derived(stateVersion && store.$getTotalSize());
 
   let indexes = $derived.by(() => {
-    const len = data.length;
+    // The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update
+    // https://svelte.dev/docs/svelte/await-expressions#Breaking-changes
+    // https://github.com/inokawa/virtua/pull/847
+    // https://github.com/inokawa/virtua/pull/1018
+    const len = stateVersion && min(data.length, layout.$getLength());
 
     const [start, end] = range;
     const arr: number[] = [];
@@ -92,17 +97,13 @@
       for (let i = start; i <= end; i++) {
         mounted.add(i);
       }
-      // The store may still have the previous length in async mode, as the range does
-      const storeLen = stateVersion && layout.$getLength();
       for (const index of sort([...mounted])) {
-        if (index < len && index < storeLen) {
+        if (index < len) {
           arr.push(index);
         }
       }
     } else {
       for (let i = start; i <= end; i++) {
-        // Guard for experimental.async: true, which runs the each block before $effect.pre
-        // https://github.com/inokawa/virtua/pull/847
         if (i < len) {
           arr.push(i);
         }
@@ -136,9 +137,7 @@
   });
 
   $effect.pre(() => {
-    if (startMargin !== store.$getStartSpacerSize()) {
-      store.$update(ACTION_START_OFFSET_CHANGE, startMargin);
-    }
+    store.$update(ACTION_START_OFFSET_CHANGE, startMargin);
   });
 
   let prevStateVersion: StateVersion | undefined;

package/lib/svelte/WindowVirtualizer.svelte

@@ -13,6 +13,7 @@
     getScrollSize as _getScrollSize,
     createWindowDriver,
     scrollToIndex as _scrollToIndex,
+    min,
   } from "../core/index.js";
   import { defaultGetKey, isSameRange, styleToString } from "./utils.js";
   import ListItem from "./ListItem.svelte";
@@ -74,13 +75,15 @@
   let totalSize = $derived(stateVersion && store.$getTotalSize());
 
   let indexes = $derived.by(() => {
-    const len = data.length;
+    // The each block runs before $effect.pre with experimental.async, so the store may still have the length before the update
+    // https://svelte.dev/docs/svelte/await-expressions#Breaking-changes
+    // https://github.com/inokawa/virtua/pull/847
+    // https://github.com/inokawa/virtua/pull/1018
+    const len = stateVersion && min(data.length, layout.$getLength());
 
     const [start, end] = range;
     const arr: number[] = [];
     for (let i = start; i <= end; i++) {
-      // Guard for experimental.async: true, which runs the each block before $effect.pre
-      // https://github.com/inokawa/virtua/pull/847
       if (i < len) {
         arr.push(i);
       }

package/lib/vue/index.cjs

@@ -347,11 +347,13 @@ t), g = e => n(e.t(), e.o()), $ = (e, t, o, r) => {
         passive: !0
     }), o.addEventListener("touchend", R, {
         passive: !0
+    }), o.addEventListener("touchcancel", R, {
+        passive: !0
     }), {
         J: k,
         X: () => {
             o.removeEventListener("scroll", k), o.removeEventListener("wheel", O), o.removeEventListener("touchstart", T), 
-            o.removeEventListener("touchend", R), a(d);
+            o.removeEventListener("touchend", R), o.removeEventListener("touchcancel", R), a(d);
         },
         D: () => {
             const [t, o] = e.C();