@homebound/beam 3.1.0 → 3.1.1

package/dist/index.js

@@ -10702,6 +10702,7 @@ function VirtualizedOptions(props) {
     // eslint-disable-next-line react-hooks/exhaustive-deps
     [focusedItem]
   );
+  const key = process.env.NODE_ENV === "test" ? items.length : "virtuoso";
   return /* @__PURE__ */ jsx50(
     Virtuoso,
     {
@@ -10709,11 +10710,6 @@ function VirtualizedOptions(props) {
       totalListHeightChanged: onListHeightChange,
       totalCount: items.length,
       ...process.env.NODE_ENV === "test" ? {
-        // In tests, we render all rows so assertions can see expands/async-loaded items. However,
-        // the `initialItemCount` (next prop) is only applied on amount, so we set `key={items.length}`
-        // to force a remount when our list changes -- and we only want/need this in tests, b/c otherwise
-        // in production a Virtuoso remount causes visible flashing.
-        key: items.length,
         // We don't really need to set this, but it's handy for tests, which would otherwise render
         // just 1 row. A better way to do this would be to jest.mock out Virtuoso with an impl that
         // just rendered everything, but doing this for now.
@@ -10756,7 +10752,8 @@ function VirtualizedOptions(props) {
       components: !loading ? {} : {
         Footer: typeof loading === "function" ? loading : () => /* @__PURE__ */ jsx50(LoadingDots, { contrast })
       }
-    }
+    },
+    key
   );
 }