@djangocfg/widget-data 0.1.7 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@djangocfg/widget-data",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Data surfaces: file tree, JSON viewer, data grid and table, kanban, listbox, masonry and timeline",
   "keywords": [
     "tree",
@@ -90,21 +90,21 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@djangocfg/widget-kit": "^0.1.7",
+    "@djangocfg/widget-kit": "^0.1.8",
     "@dnd-kit/core": "^6.3.1",
     "@radix-ui/react-direction": "^1.1.2",
     "@radix-ui/react-slot": "^1.3.0",
     "class-variance-authority": "^0.7.1"
   },
   "peerDependencies": {
-    "@djangocfg/ui-core": "^2.1.548",
+    "@djangocfg/ui-core": "^2.1.549",
     "lucide-react": "^0.545.0",
     "react": "^19.0.0",
     "react-dom": "^19.0.0"
   },
   "devDependencies": {
-    "@djangocfg/typescript-config": "^2.1.548",
-    "@djangocfg/ui-core": "^2.1.548",
+    "@djangocfg/typescript-config": "^2.1.549",
+    "@djangocfg/ui-core": "^2.1.549",
     "@storybook/react-vite": "^10.5.0",
     "@types/node": "^25.9.5",
     "@types/react": "19.2.15",

package/src/DataGrid/context/DataGridContext.tsx

@@ -262,7 +262,10 @@ export function DataGridProvider<T>(props: DataGridProviderProps<T>) {
   // Pagination
   const totalRows = filteredData.length;
   const pageCount = Math.max(1, Math.ceil(totalRows / effectivePagination.pageSize));
-  const safePage = Math.min(effectivePagination.page, pageCount);
+  // Clamped at both ends, like `setPage`. A controlled host passing a
+  // 0-indexed page makes `start` negative, and `slice(-n, 0)` renders an
+  // empty grid while the page count still claims there are rows.
+  const safePage = Math.max(1, Math.min(effectivePagination.page, pageCount));
 
   const paginatedData = useMemo(() => {
     const start = (safePage - 1) * effectivePagination.pageSize;

package/src/DataTable/context/DataTableContext.tsx

@@ -256,7 +256,10 @@ export function DataTableProvider<T>(props: DataTableProviderProps<T>) {
   // Pagination
   const totalRows = filteredData.length;
   const pageCount = Math.max(1, Math.ceil(totalRows / effectivePagination.pageSize));
-  const safePage = Math.min(effectivePagination.page, pageCount);
+  // Clamped at both ends, like `setPage`. A controlled host passing a
+  // 0-indexed page makes `start` negative, and `slice(-n, 0)` renders an
+  // empty grid while the page count still claims there are rows.
+  const safePage = Math.max(1, Math.min(effectivePagination.page, pageCount));
 
   const paginatedData = useMemo(() => {
     const start = (safePage - 1) * effectivePagination.pageSize;

package/src/JsonTree/JsonViewer.tsx

@@ -597,8 +597,13 @@ export const JsonTree = memo(function JsonTree({
     const a = document.createElement('a');
     a.href = url;
     a.download = downloadFilename;
+    // Appended, and revoked a tick later: the click is queued, so revoking in
+    // the next statement can pull the blob out from under a download that has
+    // not started reading it yet — a silent zero-byte file.
+    document.body.appendChild(a);
     a.click();
-    URL.revokeObjectURL(url);
+    a.remove();
+    setTimeout(() => URL.revokeObjectURL(url), 0);
   }, [jsonString, downloadFilename]);
 
   const toggleSearch = useCallback(() => {

package/src/Masonry/context/MasonryProvider.tsx

@@ -645,12 +645,7 @@ function useResizeObserver(positioner: Positioner) {
       return () => ({ disconnect: () => {}, observe: () => {}, unobserve: () => {} });
     }
 
-    const cache = new WeakMap<object, ResizeObserver>();
-
     return (positioner: Positioner, onUpdate: () => void) => {
-      const key = { positioner, onUpdate };
-      if (cache.has(key)) return cache.get(key)!;
-
       const updates: number[] = [];
       const itemMap = new WeakMap<Element, number>();
 
@@ -700,13 +695,18 @@ function useResizeObserver(positioner: Positioner) {
         }
       };
 
-      cache.set(key, observer);
       return observer;
     };
   }, []);
 
-  const resizeObserver = createResizeObserver(positioner, () =>
-    setLayoutVersion((prev) => prev + 1),
+  // Memoized on `positioner`. Calling the factory during render built a new
+  // ResizeObserver every time — the WeakMap that was meant to prevent it keyed
+  // on a fresh object literal, so it never hit — and each new identity made the
+  // effect below disconnect and rebuild the observer for the whole grid, on
+  // every throttled scroll tick.
+  const resizeObserver = React.useMemo(
+    () => createResizeObserver(positioner, () => setLayoutVersion((prev) => prev + 1)),
+    [createResizeObserver, positioner],
   );
 
   React.useEffect(() => () => resizeObserver.disconnect(), [resizeObserver]);