simple-table-core 3.8.9 → 3.9.1

package/dist/src/managers/AnimationCoordinator.d.ts

@@ -205,6 +205,73 @@ export declare class AnimationCoordinator {
      * sort/reorder FLIP-from-off-screen behavior is preserved.
      */
     hasSnapshotEntryInContainer(cellId: string, currentContainer: HTMLElement): boolean;
+    /**
+     * Whether a vertical position transition should be animated. Rows that live
+     * only in the virtualization padding band (above/below the visible viewport)
+     * should teleport rather than FLIP — otherwise a mid-scroll sort animates
+     * hundreds of off-screen rows through the viewport at once.
+     */
+    shouldAnimateVerticalTransition(args: {
+        beforeTop: number;
+        afterTop: number;
+        cellHeight: number;
+        container: HTMLElement;
+    }): boolean;
+    /**
+     * Whether a horizontal position transition should be animated. Same viewport
+     * gate as {@link shouldAnimateVerticalTransition} but for column slides.
+     */
+    shouldAnimateHorizontalTransition(args: {
+        beforeLeft: number;
+        afterLeft: number;
+        cellWidth: number;
+        container: HTMLElement;
+    }): boolean;
+    /**
+     * Whether a cell's position change should participate in FLIP animation.
+     * Only axes that actually moved are checked against the viewport gate —
+     * a vertical sort must not inherit "visible" from an unchanged horizontal
+     * position (which would retain/mount every column in every padding-band row).
+     */
+    shouldAnimateTransition(args: {
+        beforeTop: number;
+        afterTop: number;
+        beforeLeft: number;
+        afterLeft: number;
+        cellHeight: number;
+        cellWidth: number;
+        container: HTMLElement;
+    }): boolean;
+    /**
+     * Whether an incoming cell (in the snapshot but not previously in the DOM)
+     * should be mounted for this sort/reorder render. Padding-band rows that
+     * never intersect the visible viewport are deferred to the next scroll
+     * render so they do not pop into existence at animation start.
+     */
+    shouldMountIncomingCell(args: {
+        cellId: string;
+        afterTop: number;
+        afterLeft: number;
+        cellHeight: number;
+        cellWidth: number;
+        container: HTMLElement;
+    }): boolean;
+    /**
+     * Whether the cell currently paints inside the body scrollers' clip rects.
+     * Outgoing retain decisions use this as a fallback when {@link shouldAnimateTransition}
+     * rejects a cell based on `style.top`/`style.left` alone: virtualization padding
+     * rows can sit with their leading edge outside the scroll band while still being
+     * partially visible on screen (common at the bottom when scrolled to the end).
+     */
+    isCellRenderedInScrollerViewport(element: HTMLElement, container: HTMLElement): boolean;
+    /**
+     * Whether an outgoing DOM cell at a vertical scroll extreme (top or bottom of
+     * the dataset) should be retained even when {@link shouldAnimateTransition}
+     * rejects it on leading-edge grounds. Virtualization keeps a few overscan
+     * rows mounted above/below the strict viewport at max scroll; those rows
+     * must still slide out as ghosts when a sort evicts them.
+     */
+    shouldRetainDomCellAtScrollExtrema(cellId: string, container: HTMLElement): boolean;
     /**
      * Hand a cell that the renderer would otherwise remove to the coordinator.
      * The coordinator updates its absolute positioning to the post-change layout

package/dist/src/managers/DimensionManager.d.ts

@@ -21,12 +21,20 @@ export interface DimensionManagerState {
     contentHeight: number | undefined;
 }
 type StateChangeCallback = (state: DimensionManagerState) => void;
+/**
+ * Wait this long after the last container-width ResizeObserver tick before
+ * notifying subscribers. Coalesces CSS-transition / animated layout shifts
+ * (e.g. a collapsible nav) into a single trailing update instead of one full
+ * table render per animation frame.
+ */
+export declare const CONTAINER_RESIZE_NOTIFY_DEBOUNCE_MS = 50;
 export declare class DimensionManager {
     private config;
     private state;
     private subscribers;
     private resizeObserver;
     private rafId;
+    private resizeNotifyDebounceId;
     /** Set when applyContainerWidthSync updates state before any subscriber exists. */
     private initialNotifyPending;
     constructor(config: DimensionManagerConfig);
@@ -35,6 +43,9 @@ export declare class DimensionManager {
     private calculateHeaderHeight;
     private convertHeightToPixels;
     private calculateContentHeight;
+    private cancelPendingResizeNotify;
+    /** Trailing debounce for animated / continuous container resizes. */
+    private scheduleResizeNotify;
     private observeContainer;
     private applyContainerWidthSync;
     updateConfig(config: Partial<DimensionManagerConfig>): void;

package/dist/src/types/AnimationsConfig.d.ts

@@ -3,6 +3,8 @@
  *
  * The animation coordinator runs FLIP-style transitions when cells move between
  * positions. All fields are optional; omit the prop entirely to use defaults.
+ *
+ * Row group expand/collapse never animates when `rowGrouping` is configured.
  */
 export interface AnimationsConfig {
     /** Master toggle. Defaults to `true`. When `false`, no other field has effect. */

package/dist/stories/tests/43-CollapseExpandAnimationsTests.stories.d.ts

@@ -34,7 +34,7 @@ export declare const ColumnExpand_IncomingCellsStartAtZeroWidth: {
         canvasElement: HTMLElement;
     }) => Promise<void>;
 };
-export declare const RowExpand_AddsAccordionClass: {
+export declare const RowExpand_NoAccordionClass: {
     tags: string[];
     render: () => HTMLDivElement & {
         _table?: import("../../src/index").SimpleTableVanilla | undefined;
@@ -43,7 +43,7 @@ export declare const RowExpand_AddsAccordionClass: {
         canvasElement: HTMLElement;
     }) => Promise<void>;
 };
-export declare const RowExpand_IncomingCellsStartAtZeroHeight: {
+export declare const RowExpand_NoAccordionGrow: {
     tags: string[];
     render: () => HTMLDivElement & {
         _table?: import("../../src/index").SimpleTableVanilla | undefined;

package/dist/stories/tests/50-ContainerResizePerformanceTests.stories.d.ts

@@ -0,0 +1,56 @@
+/**
+ * CONTAINER RESIZE PERFORMANCE REPRO
+ *
+ * Simulates a collapsible global left nav that animates width and squeezes the
+ * main content area — the layout pattern reported on Artist/Platform pages.
+ *
+ * During the CSS transition, the table container's ResizeObserver fires on
+ * successive frames. DimensionManager coalesces those ticks (trailing debounce)
+ * so subscribers / full renders fire once after the transition settles.
+ *
+ * Use the HUD while toggling the nav, or run the automated play test.
+ * The play test asserts coalesced table relayouts (header width updates), not
+ * raw container ResizeObserver ticks — the DOM still resizes every animation frame.
+ */
+import type { Meta } from "@storybook/html";
+declare const meta: Meta;
+export default meta;
+interface ContainerResizePerfSnapshot {
+    contentWidthChanges: number;
+    tableContainerWidthChanges: number;
+    /** Batched header relayout passes (one per table render), not per-cell updates. */
+    tableRelayouts: number;
+    lastContentWidth: number;
+}
+declare global {
+    interface Window {
+        __containerResizePerf?: ContainerResizePerfSnapshot;
+    }
+}
+export declare const resetContainerResizePerf: (lastContentWidth?: number) => void;
+export declare const CollapsibleNavWithTable: {
+    tags: string[];
+    render: () => HTMLDivElement;
+};
+export declare const CollapsibleNavTableNoAutoExpand: {
+    tags: string[];
+    render: () => HTMLDivElement;
+};
+export declare const CollapsibleNavBaseline: {
+    tags: string[];
+    render: () => HTMLDivElement;
+};
+export declare const NavToggleCoalescesTableRelayout: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+export declare const BaselineNavHasNoTableGridChurn: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};

package/dist/stories/tests/51-RowGroupReExpandTests.stories.d.ts

@@ -0,0 +1,78 @@
+/**
+ * ROW GROUP RE-EXPAND REGRESSION TESTS
+ *
+ * Reproduces bugs when a lazy-loaded grouped row is collapsed and re-expanded:
+ *
+ *   1. onRowGroupExpand receives a stale `row` snapshot (missing cached children),
+ *      so consumer guards like `if (row[groupingKey]?.length) return` fail and
+ *      every re-expand re-triggers setLoading + fetch.
+ *
+ *   2. Re-expand must not re-enter loading state or spuriously animate sibling
+ *      rows below the expanded group.
+ *
+ * Mirrors packages/react/src/__tests__/onRowGroupExpandReExpand.test.tsx.
+ * All play tests should FAIL until the stale-row fix lands.
+ */
+import type { Meta } from "@storybook/html";
+declare const meta: Meta;
+export default meta;
+interface ReExpandProbe {
+    expandSnapshots: Array<{
+        childCount: number;
+        isExpanded: boolean;
+    }>;
+    fetchCount: number;
+    loadingTrueCount: number;
+}
+declare global {
+    interface Window {
+        __rowGroupReExpandProbe?: ReExpandProbe;
+    }
+}
+export declare const ReExpand_PassesCachedRowSnapshot: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+export declare const ReExpand_SkipsRefetch: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+export declare const ReExpand_SkipsSetLoading: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+/** Manual QA: status panel shows fetch/loading counts while toggling Engineering. */
+export declare const ReExpand_LazyLoad_StatusPanel: {
+    tags: string[];
+    render: () => HTMLDivElement;
+};
+export declare const ReExpand_NoLoadingSkeletonOnLazyReExpand: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+export declare const ReExpand_SiblingRowsNoAccordionGrow: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};
+export declare const ReExpand_NoAccordionGrowOnReExpand: {
+    tags: string[];
+    render: () => HTMLDivElement;
+    play: ({ canvasElement }: {
+        canvasElement: HTMLElement;
+    }) => Promise<void>;
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "simple-table-core",
-  "version": "3.8.9",
+  "version": "3.9.1",
   "main": "dist/cjs/index.js",
   "module": "dist/index.es.js",
   "types": "dist/src/index.d.ts",