react-arborist 3.7.0 → 3.8.0

package/dist/module/components/cursor.js

@@ -8,8 +8,7 @@ export function Cursor() {
     if (!cursor || cursor.type !== "line")
         return null;
     const indent = tree.indent;
-    const top = tree.rowHeight * cursor.index +
-        ((_b = (_a = tree.props.padding) !== null && _a !== void 0 ? _a : tree.props.paddingTop) !== null && _b !== void 0 ? _b : 0);
+    const top = tree.rowTopPosition(cursor.index) + ((_b = (_a = tree.props.padding) !== null && _a !== void 0 ? _a : tree.props.paddingTop) !== null && _b !== void 0 ? _b : 0);
     const left = indent * cursor.level;
     const Cursor = tree.renderCursor;
     return _jsx(Cursor, { top, left, indent });

package/dist/module/components/default-container.js

@@ -1,5 +1,5 @@
 import { jsx as _jsx } from "react/jsx-runtime";
-import { FixedSizeList } from "react-window";
+import { FixedSizeList, VariableSizeList } from "react-window";
 import { useDataUpdates, useTreeApi } from "../context";
 import { focusNextElement, focusPrevElement } from "../utils";
 import { ListOuterElement } from "./list-outer-element";
@@ -13,7 +13,6 @@ let timeoutId = null;
  * the event handler. Future clean up welcome.
  */
 export function DefaultContainer() {
-    var _a, _b;
     useDataUpdates();
     const tree = useTreeApi();
     return (_jsx("div", { role: "tree", style: {
@@ -231,5 +230,35 @@ export function DefaultContainer() {
             });
             if (node)
                 tree.focus(node.id);
-        }, children: _jsx(FixedSizeList, { className: tree.props.className, outerRef: tree.listEl, itemCount: tree.visibleNodes.length, height: tree.height, width: tree.width, itemSize: tree.rowHeight, overscanCount: tree.overscanCount, itemKey: (index) => { var _a; return ((_a = tree.visibleNodes[index]) === null || _a === void 0 ? void 0 : _a.id) || index; }, outerElementType: (_a = tree.props.outerElementType) !== null && _a !== void 0 ? _a : ListOuterElement, innerElementType: (_b = tree.props.innerElementType) !== null && _b !== void 0 ? _b : ListInnerElement, onScroll: tree.props.onScroll, onItemsRendered: tree.onItemsRendered.bind(tree), ref: tree.list, children: RowContainer }) }));
+        }, children: _jsx(List, {}) }));
+}
+/**
+ * Fixed-height trees (numeric rowHeight) render a FixedSizeList, preserving the
+ * original O(1) layout and avoiding VariableSizeList's measurement cache. Only
+ * the function form, which needs per-row heights, uses VariableSizeList.
+ */
+function List() {
+    var _a, _b;
+    const tree = useTreeApi();
+    const commonProps = {
+        className: tree.props.className,
+        outerRef: tree.listEl,
+        itemCount: tree.visibleNodes.length,
+        height: tree.height,
+        width: tree.width,
+        overscanCount: tree.overscanCount,
+        itemKey: (index) => { var _a; return ((_a = tree.visibleNodes[index]) === null || _a === void 0 ? void 0 : _a.id) || index; },
+        outerElementType: (_a = tree.props.outerElementType) !== null && _a !== void 0 ? _a : ListOuterElement,
+        innerElementType: (_b = tree.props.innerElementType) !== null && _b !== void 0 ? _b : ListInnerElement,
+        onScroll: tree.props.onScroll,
+        onItemsRendered: tree.onItemsRendered.bind(tree),
+    };
+    if (typeof tree.props.rowHeight === "function") {
+        return (
+        // @ts-ignore
+        _jsx(VariableSizeList, Object.assign({}, commonProps, { itemSize: tree.rowHeightAt, ref: tree.list, children: RowContainer })));
+    }
+    return (
+    // @ts-ignore
+    _jsx(FixedSizeList, Object.assign({}, commonProps, { itemSize: tree.rowHeight, ref: tree.list, children: RowContainer })));
 }

package/dist/module/components/default-cursor.js

@@ -17,7 +17,7 @@ const circleStyle = {
     boxShadow: "0 0 0 3px #4B91E2",
     borderRadius: "50%",
 };
-export const DefaultCursor = React.memo(function DefaultCursor({ top, left, indent, }) {
+export const DefaultCursor = React.memo(function DefaultCursor({ top, left, indent }) {
     const style = {
         position: "absolute",
         pointerEvents: "none",

package/dist/module/components/default-drag-preview.d.ts

@@ -1,2 +1,2 @@
 import { DragPreviewProps } from "../types/renderers";
-export declare function DefaultDragPreview({ offset, mouse, id, dragIds, isDragging, }: DragPreviewProps): import("react/jsx-runtime").JSX.Element;
+export declare function DefaultDragPreview({ offset, mouse, id, dragIds, isDragging }: DragPreviewProps): import("react/jsx-runtime").JSX.Element;

package/dist/module/components/default-drag-preview.js

@@ -22,7 +22,7 @@ const getCountStyle = (offset) => {
     const { x, y } = offset;
     return { transform: `translate(${x + 10}px, ${y + 10}px)` };
 };
-export function DefaultDragPreview({ offset, mouse, id, dragIds, isDragging, }) {
+export function DefaultDragPreview({ offset, mouse, id, dragIds, isDragging }) {
     return (_jsxs(Overlay, { isDragging: isDragging, children: [_jsx(Position, { offset: offset, children: _jsx(PreviewNode, { id: id, dragIds: dragIds }) }), _jsx(Count, { mouse: mouse, count: dragIds.length })] }));
 }
 const Overlay = memo(function Overlay(props) {

package/dist/module/components/default-row.d.ts

@@ -1,2 +1,2 @@
 import { RowRendererProps } from "../types/renderers";
-export declare function DefaultRow<T>({ node, attrs, innerRef, children, }: RowRendererProps<T>): import("react/jsx-runtime").JSX.Element;
+export declare function DefaultRow<T>({ node, attrs, innerRef, children }: RowRendererProps<T>): import("react/jsx-runtime").JSX.Element;

package/dist/module/components/default-row.js

@@ -1,4 +1,4 @@
 import { jsx as _jsx } from "react/jsx-runtime";
-export function DefaultRow({ node, attrs, innerRef, children, }) {
+export function DefaultRow({ node, attrs, innerRef, children }) {
     return (_jsx("div", Object.assign({}, attrs, { ref: innerRef, onFocus: (e) => e.stopPropagation(), onClick: node.handleClick, children: children })));
 }

package/dist/module/components/list-outer-element.js

@@ -26,7 +26,7 @@ export const ListOuterElement = forwardRef(function Outer(props, ref) {
 export const DropContainer = () => {
     const tree = useTreeApi();
     return (_jsx("div", { style: {
-            height: tree.visibleNodes.length * tree.rowHeight,
+            height: tree.rowTopPosition(tree.visibleNodes.length),
             width: "100%",
             position: "absolute",
             left: "0",

package/dist/module/components/provider.d.ts

@@ -6,5 +6,5 @@ type Props<T> = {
     imperativeHandle: React.Ref<TreeApi<T> | undefined>;
     children: ReactNode;
 };
-export declare function TreeProvider<T>({ treeProps, imperativeHandle, children, }: Props<T>): import("react/jsx-runtime").JSX.Element;
+export declare function TreeProvider<T>({ treeProps, imperativeHandle, children }: Props<T>): import("react/jsx-runtime").JSX.Element;
 export {};

package/dist/module/components/provider.js

@@ -1,7 +1,7 @@
 import { jsx as _jsx } from "react/jsx-runtime";
-import { useEffect, useImperativeHandle, useMemo, useRef, } from "react";
+import { useEffect, useImperativeHandle, useMemo, useRef } from "react";
 import { useSyncExternalStore } from "use-sync-external-store/shim";
-import { DataUpdatesContext, DndContext, NodesContext, TreeApiContext, } from "../context";
+import { DataUpdatesContext, DndContext, NodesContext, TreeApiContext } from "../context";
 import { TreeApi } from "../interfaces/tree-api";
 import { initialState } from "../state/initial";
 import { rootReducer } from "../state/root-reducer";
@@ -10,7 +10,7 @@ import { DndProvider } from "react-dnd";
 import { createStore } from "redux";
 import { actions as visibility } from "../state/open-slice";
 const SERVER_STATE = initialState();
-export function TreeProvider({ treeProps, imperativeHandle, children, }) {
+export function TreeProvider({ treeProps, imperativeHandle, children }) {
     const list = useRef(null);
     const listEl = useRef(null);
     const store = useRef(
@@ -26,7 +26,7 @@ export function TreeProvider({ treeProps, imperativeHandle, children, }) {
     useMemo(() => {
         updateCount.current += 1;
         api.update(treeProps);
-    }, [...Object.values(treeProps)]);
+    }, Object.values(treeProps));
     /* Rebuild visible nodes when open state changes, without clobbering
        props set imperatively via api.update(). Bumping updateCount keeps
        DataUpdates consumers (e.g. DefaultContainer) in sync. */

package/dist/module/components/provider.test.js

@@ -1,6 +1,7 @@
 import { jsx as _jsx } from "react/jsx-runtime";
 import { createRef } from "react";
-import { act, render } from "@testing-library/react";
+import { act, render, screen } from "@testing-library/react";
+import { FixedSizeList, VariableSizeList } from "react-window";
 import { Tree } from "./tree";
 const data = [
     {
@@ -29,3 +30,72 @@ test("imperative tree.update() props survive node toggles (#228)", () => {
     });
     expect(api.rowHeight).toBe(48);
 });
+/* Backwards compatibility: switching FixedSizeList -> VariableSizeList must not
+   change layout for a numeric rowHeight. With openByDefault, all four nodes
+   (1 > 2, 3 > 4) are visible in DFS order. */
+test("numeric rowHeight positions rows at index * height (#238 back-compat)", () => {
+    render(_jsx(Tree, { data: data, rowHeight: 24, openByDefault: true }));
+    const rows = screen.getAllByRole("treeitem");
+    expect(rows).toHaveLength(4);
+    rows.forEach((row, i) => {
+        expect(row.style.height).toBe("24px");
+        expect(row.style.top).toBe(`${i * 24}px`);
+    });
+});
+test("function rowHeight gives each row its own height and cumulative top (#238)", () => {
+    const heights = { "1": 40, "2": 20, "3": 30, "4": 10 };
+    render(_jsx(Tree, { data: data, rowHeight: (node) => heights[node.id], openByDefault: true }));
+    const rows = screen.getAllByRole("treeitem");
+    expect(rows).toHaveLength(4);
+    const expected = [40, 20, 30, 10];
+    let top = 0;
+    rows.forEach((row, i) => {
+        expect(row.style.height).toBe(`${expected[i]}px`);
+        expect(row.style.top).toBe(`${top}px`);
+        top += expected[i];
+    });
+});
+test("mutations tell the list to recompute heights (#238)", () => {
+    const ref = createRef();
+    /* Only variable-height mode renders a VariableSizeList with a measurement
+       cache to recompute, so use a function rowHeight here. */
+    render(_jsx(Tree, { data: data, ref: ref, rowHeight: () => 24, openByDefault: true }));
+    const api = ref.current;
+    const reset = jest.spyOn(api.list.current, "resetAfterIndex");
+    act(() => api.close("1"));
+    expect(reset).toHaveBeenCalled();
+    reset.mockClear();
+    act(() => api.open("1"));
+    expect(reset).toHaveBeenCalled();
+});
+/* react-window caches measurements by index and never invalidates them itself.
+   When data changes via props in variable-height mode, those cached sizes belong
+   to the wrong rows, so update() must drop the cache. It runs during render, so
+   it uses the shouldForceUpdate=false variant. */
+test("changing data in variable-height mode resets the list cache (#238)", () => {
+    const ref = createRef();
+    const rowHeight = (node) => (node.isInternal ? 40 : 20);
+    const { rerender } = render(_jsx(Tree, { data: data, ref: ref, rowHeight: rowHeight, openByDefault: true }));
+    const reset = jest.spyOn(ref.current.list.current, "resetAfterIndex");
+    const nextData = [{ id: "9", name: "fresh" }, ...data];
+    act(() => {
+        rerender(_jsx(Tree, { data: nextData, ref: ref, rowHeight: rowHeight, openByDefault: true }));
+    });
+    expect(reset).toHaveBeenCalledWith(0, false);
+});
+/* The numeric path must stay on FixedSizeList: it has constant item sizes, so
+   there is no measurement cache to go stale and none of VariableSizeList's
+   overhead. A FixedSizeList has no resetAfterIndex method at all. */
+test("numeric rowHeight renders a cache-free FixedSizeList (#238)", () => {
+    const ref = createRef();
+    render(_jsx(Tree, { data: data, ref: ref, rowHeight: 24, openByDefault: true }));
+    const list = ref.current.list.current;
+    expect(list).toBeInstanceOf(FixedSizeList);
+    expect("resetAfterIndex" in list).toBe(false);
+});
+/* The function path uses VariableSizeList so per-row heights are possible. */
+test("function rowHeight renders a VariableSizeList (#238)", () => {
+    const ref = createRef();
+    render(_jsx(Tree, { data: data, ref: ref, rowHeight: () => 24, openByDefault: true }));
+    expect(ref.current.list.current).toBeInstanceOf(VariableSizeList);
+});

package/dist/module/components/row-container.d.ts

@@ -3,5 +3,5 @@ type Props = {
     style: React.CSSProperties;
     index: number;
 };
-export declare const RowContainer: React.MemoExoticComponent<(<T>({ index, style, }: Props) => import("react/jsx-runtime").JSX.Element)>;
+export declare const RowContainer: React.MemoExoticComponent<(<T>({ index, style }: Props) => import("react/jsx-runtime").JSX.Element)>;
 export {};

package/dist/module/components/row-container.js

@@ -4,7 +4,7 @@ import { useDataUpdates, useNodesContext, useTreeApi } from "../context";
 import { useDragHook } from "../dnd/drag-hook";
 import { useDropHook } from "../dnd/drop-hook";
 import { useFreshNode } from "../hooks/use-fresh-node";
-export const RowContainer = React.memo(function RowContainer({ index, style, }) {
+export const RowContainer = React.memo(function RowContainer({ index, style }) {
     /* When will the <Row> will re-render.
      *
      * The row component is memo'd so it will only render
@@ -35,8 +35,7 @@ export const RowContainer = React.memo(function RowContainer({ index, style, })
     const nodeStyle = useMemo(() => ({ paddingLeft: indent }), [indent]);
     const rowStyle = useMemo(() => {
         var _a, _b;
-        return (Object.assign(Object.assign({}, style), { top: parseFloat(style.top) +
-                ((_b = (_a = tree.props.padding) !== null && _a !== void 0 ? _a : tree.props.paddingTop) !== null && _b !== void 0 ? _b : 0) }));
+        return (Object.assign(Object.assign({}, style), { top: parseFloat(style.top) + ((_b = (_a = tree.props.padding) !== null && _a !== void 0 ? _a : tree.props.paddingTop) !== null && _b !== void 0 ? _b : 0) }));
     }, [style, tree.props.padding, tree.props.paddingTop]);
     const rowAttrs = {
         role: "treeitem",

package/dist/module/dnd/compute-drop.js

@@ -1,4 +1,4 @@
-import { bound, indexOf, isClosed, isItem, isOpenWithEmptyChildren, } from "../utils";
+import { bound, indexOf, isClosed, isItem, isOpenWithEmptyChildren } from "../utils";
 function measureHover(el, offset) {
     const rect = el.getBoundingClientRect();
     const x = offset.x - Math.round(rect.x);

package/dist/module/dnd/drag-hook.js

@@ -17,6 +17,7 @@ export function useDragHook(node) {
         },
         end: () => {
             tree.hideCursor();
+            tree.redrawList();
             tree.dispatch(dnd.dragEnd());
         },
     }), [ids, node]);

package/dist/module/hooks/use-validated-props.js

@@ -3,8 +3,7 @@ export function useValidatedProps(props) {
     if (props.initialData && props.data) {
         throw new Error(`React Arborist Tree => Provide either a data or initialData prop, but not both.`);
     }
-    if (props.initialData &&
-        (props.onCreate || props.onDelete || props.onMove || props.onRename)) {
+    if (props.initialData && (props.onCreate || props.onDelete || props.onMove || props.onRename)) {
         throw new Error(`React Arborist Tree => You passed the initialData prop along with a data handler.
 Use the data prop if you want to provide your own handlers.`);
     }

package/dist/module/interfaces/node-api.js

@@ -3,7 +3,10 @@ export class NodeApi {
     constructor(params) {
         this.handleClick = (e) => {
             if (e.metaKey && !this.tree.props.disableMultiSelection) {
-                this.isSelected ? this.deselect() : this.selectMulti();
+                if (this.isSelected)
+                    this.deselect();
+                else
+                    this.selectMulti();
             }
             else if (e.shiftKey && !this.tree.props.disableMultiSelection) {
                 this.selectContiguous();

package/dist/module/interfaces/tree-api.d.ts

@@ -2,7 +2,7 @@ import { EditResult } from "../types/handlers";
 import { Identity, IdObj } from "../types/utils";
 import { TreeProps } from "../types/tree-props";
 import { MutableRefObject } from "react";
-import { Align, FixedSizeList, ListOnItemsRenderedProps } from "react-window";
+import { Align, FixedSizeList, ListOnItemsRenderedProps, VariableSizeList } from "react-window";
 import { DefaultRow } from "../components/default-row";
 import { DefaultNode } from "../components/default-node";
 import { NodeApi } from "./node-api";
@@ -14,7 +14,7 @@ import { Store } from "redux";
 export declare class TreeApi<T> {
     store: Store<RootState, Actions>;
     props: TreeProps<T>;
-    list: MutableRefObject<FixedSizeList | null>;
+    list: MutableRefObject<FixedSizeList | VariableSizeList | null>;
     listEl: MutableRefObject<HTMLDivElement | null>;
     static editPromise: null | ((args: EditResult) => void);
     root: NodeApi<T>;
@@ -24,7 +24,8 @@ export declare class TreeApi<T> {
     idToIndex: {
         [id: string]: number;
     };
-    constructor(store: Store<RootState, Actions>, props: TreeProps<T>, list: MutableRefObject<FixedSizeList | null>, listEl: MutableRefObject<HTMLDivElement | null>);
+    private rowOffsets;
+    constructor(store: Store<RootState, Actions>, props: TreeProps<T>, list: MutableRefObject<FixedSizeList | VariableSizeList | null>, listEl: MutableRefObject<HTMLDivElement | null>);
     update(props: TreeProps<T>): void;
     dispatch(action: Actions): {
         type: "FOCUS";
@@ -121,7 +122,28 @@ export declare class TreeApi<T> {
     get width(): string | number;
     get height(): number;
     get indent(): number;
+    /**
+     * The fixed row height. When a `rowHeight` function is supplied for variable
+     * heights, this returns the default (24); use `rowHeightAt(index)` to get the
+     * height of a specific row.
+     */
     get rowHeight(): number;
+    /**
+     * The height of the row at `index`, evaluating the `rowHeight` function if
+     * given. Falls back to the default height for an out-of-range index so this
+     * never feeds an invalid `0` to react-window's `itemSize`.
+     */
+    rowHeightAt: (index: number) => number;
+    /** The pixel offset of the top of the row at `index` from the top of the list. */
+    rowTopPosition: (index: number) => number;
+    /**
+     * Tell the underlying virtualized list to recompute row heights at and after
+     * `index`. Call this if a `rowHeight` function's output changes for reasons
+     * the tree can't observe (e.g. external state).
+     */
+    redrawList: (afterIndex?: number) => void;
+    /** Lazily-built prefix sum where offsets[i] is the top of row i. */
+    private getRowOffsets;
     get overscanCount(): number;
     get searchTerm(): string;
     get matchFn(): (node: NodeApi<T>) => boolean;
@@ -185,8 +207,8 @@ export declare class TreeApi<T> {
     canDrop(): boolean;
     hideCursor(): void;
     showCursor(cursor: Cursor): void;
-    open(identity: Identity): void;
-    close(identity: Identity): void;
+    open(identity: Identity, redraw?: boolean): void;
+    close(identity: Identity, redraw?: boolean): void;
     toggle(identity: Identity): void;
     openParents(identity: Identity): void;
     openSiblings(node: NodeApi<T>): void;

package/dist/module/interfaces/tree-api.js

@@ -30,6 +30,46 @@ export class TreeApi {
         this.listEl = listEl;
         this.visibleStartIndex = 0;
         this.visibleStopIndex = 0;
+        /* Memoized prefix-sum of row heights; only used for variable heights. */
+        this.rowOffsets = null;
+        /**
+         * The height of the row at `index`, evaluating the `rowHeight` function if
+         * given. Falls back to the default height for an out-of-range index so this
+         * never feeds an invalid `0` to react-window's `itemSize`.
+         */
+        this.rowHeightAt = (index) => {
+            const rowHeight = this.props.rowHeight;
+            if (typeof rowHeight === "function") {
+                const node = this.at(index);
+                return node ? rowHeight(node) : this.rowHeight;
+            }
+            return rowHeight !== null && rowHeight !== void 0 ? rowHeight : 24;
+        };
+        /** The pixel offset of the top of the row at `index` from the top of the list. */
+        this.rowTopPosition = (index) => {
+            /* Fixed heights: O(1). */
+            if (typeof this.props.rowHeight !== "function") {
+                return index * this.rowHeight;
+            }
+            /* Variable heights: O(1) amortized via a memoized prefix sum. */
+            const offsets = this.getRowOffsets();
+            const clamped = Math.max(0, Math.min(index, offsets.length - 1));
+            return offsets[clamped];
+        };
+        /**
+         * Tell the underlying virtualized list to recompute row heights at and after
+         * `index`. Call this if a `rowHeight` function's output changes for reasons
+         * the tree can't observe (e.g. external state).
+         */
+        this.redrawList = (afterIndex = 0) => {
+            this.rowOffsets = null;
+            /* Only the VariableSizeList (function rowHeight) caches measurements; a
+               FixedSizeList has constant heights and nothing to recompute. */
+            const list = this.list.current;
+            if (list && "resetAfterIndex" in list) {
+                list.resetAfterIndex(Math.max(0, afterIndex));
+            }
+        };
         /* Changes here must also be made in update() */
         this.root = createRoot(this);
         this.visibleNodes = createList(this);
@@ -41,6 +81,18 @@ export class TreeApi {
         this.root = createRoot(this);
         this.visibleNodes = createList(this);
         this.idToIndex = createIndex(this.visibleNodes);
+        this.rowOffsets = null;
+        /* Variable-height mode renders a VariableSizeList, which caches item
+           measurements by index and never invalidates them on its own. When the
+           visible nodes change (insert/remove/reorder), those cached sizes belong
+           to the wrong rows, so drop them. Fixed-height mode renders a
+           FixedSizeList (no cache, nothing to reset). update() runs during render,
+           so pass shouldForceUpdate=false: the in-progress render repaints the list
+           and a forceUpdate here would warn about setting state mid-render. */
+        const list = this.list.current;
+        if (list && "resetAfterIndex" in list) {
+            list.resetAfterIndex(0, false);
+        }
     }
     /* Store helpers */
     dispatch(action) {
@@ -65,9 +117,24 @@ export class TreeApi {
         var _a;
         return (_a = this.props.indent) !== null && _a !== void 0 ? _a : 24;
     }
+    /**
+     * The fixed row height. When a `rowHeight` function is supplied for variable
+     * heights, this returns the default (24); use `rowHeightAt(index)` to get the
+     * height of a specific row.
+     */
     get rowHeight() {
-        var _a;
-        return (_a = this.props.rowHeight) !== null && _a !== void 0 ? _a : 24;
+        return typeof this.props.rowHeight === "number" ? this.props.rowHeight : 24;
+    }
+    /** Lazily-built prefix sum where offsets[i] is the top of row i. */
+    getRowOffsets() {
+        if (this.rowOffsets)
+            return this.rowOffsets;
+        const offsets = [0];
+        for (let i = 0; i < this.visibleNodes.length; i++) {
+            offsets.push(offsets[i] + this.rowHeightAt(i));
+        }
+        this.rowOffsets = offsets;
+        return offsets;
     }
     get overscanCount() {
         var _a;
@@ -169,9 +236,7 @@ export class TreeApi {
     create() {
         return __awaiter(this, arguments, void 0, function* (opts = {}) {
             var _a, _b;
-            const parentId = opts.parentId === undefined
-                ? utils.getInsertParentId(this)
-                : opts.parentId;
+            const parentId = opts.parentId === undefined ? utils.getInsertParentId(this) : opts.parentId;
             const index = (_a = opts.index) !== null && _a !== void 0 ? _a : utils.getInsertIndex(this);
             const type = (_b = opts.type) !== null && _b !== void 0 ? _b : "leaf";
             const data = yield safeRun(this.props.onCreate, {
@@ -198,20 +263,26 @@ export class TreeApi {
             const idents = Array.isArray(node) ? node : [node];
             const ids = idents.map(identify);
             const nodes = ids.map((id) => this.get(id)).filter((n) => !!n);
+            /* Guard against Math.min(...[]) === Infinity when no ids resolve to nodes. */
+            const fromIndex = nodes.length ? Math.min(...nodes.map((n) => { var _a; return (_a = n.rowIndex) !== null && _a !== void 0 ? _a : 0; })) : 0;
             yield safeRun(this.props.onDelete, { nodes, ids });
+            this.redrawList(fromIndex);
         });
     }
     edit(node) {
+        var _a, _b;
         const id = identify(node);
         this.resolveEdit({ cancelled: true });
         this.scrollTo(id);
         this.dispatch(edit(id));
+        this.redrawList((_b = (_a = this.get(id)) === null || _a === void 0 ? void 0 : _a.rowIndex) !== null && _b !== void 0 ? _b : 0);
         return new Promise((resolve) => {
             TreeApi.editPromise = resolve;
         });
     }
     submit(identity, value) {
         return __awaiter(this, void 0, void 0, function* () {
+            var _a, _b;
             if (!identity)
                 return;
             const id = identify(identity);
@@ -222,12 +293,14 @@ export class TreeApi {
             });
             this.dispatch(edit(null));
             this.resolveEdit({ cancelled: false, value });
+            this.redrawList((_b = (_a = this.get(id)) === null || _a === void 0 ? void 0 : _a.rowIndex) !== null && _b !== void 0 ? _b : 0);
             setTimeout(() => this.onFocus()); // Return focus to element;
         });
     }
     reset() {
         this.dispatch(edit(null));
         this.resolveEdit({ cancelled: true });
+        this.redrawList();
         setTimeout(() => this.onFocus()); // Return focus to element;
     }
     activate(id) {
@@ -407,9 +480,7 @@ export class TreeApi {
         return this.state.dnd.cursor.type === "highlight";
     }
     get dragNodes() {
-        return this.state.dnd.dragIds
-            .map((id) => this.get(id))
-            .filter((n) => !!n);
+        return this.state.dnd.dragIds.map((id) => this.get(id)).filter((n) => !!n);
     }
     get dragNode() {
         return this.get(this.state.nodes.drag.id);
@@ -461,22 +532,28 @@ export class TreeApi {
         this.dispatch(dnd.cursor(cursor));
     }
     /* Visibility */
-    open(identity) {
+    open(identity, redraw = true) {
+        var _a, _b;
         const id = identifyNull(identity);
         if (!id)
             return;
         if (this.isOpen(id))
             return;
         this.dispatch(visibility.open(id, this.isFiltered));
+        if (redraw)
+            this.redrawList((_b = (_a = this.get(id)) === null || _a === void 0 ? void 0 : _a.rowIndex) !== null && _b !== void 0 ? _b : 0);
         safeRun(this.props.onToggle, id);
     }
-    close(identity) {
+    close(identity, redraw = true) {
+        var _a, _b;
         const id = identifyNull(identity);
         if (!id)
             return;
         if (!this.isOpen(id))
             return;
         this.dispatch(visibility.close(id, this.isFiltered));
+        if (redraw)
+            this.redrawList((_b = (_a = this.get(id)) === null || _a === void 0 ? void 0 : _a.rowIndex) !== null && _b !== void 0 ? _b : 0);
         safeRun(this.props.onToggle, id);
     }
     toggle(identity) {
@@ -492,9 +569,10 @@ export class TreeApi {
         const node = utils.dfs(this.root, id);
         let parent = node === null || node === void 0 ? void 0 : node.parent;
         while (parent) {
-            this.open(parent.id);
+            this.open(parent.id, false);
             parent = parent.parent;
         }
+        this.redrawList();
     }
     openSiblings(node) {
         const parent = node.parent;
@@ -505,23 +583,29 @@ export class TreeApi {
             const isOpen = node.isOpen;
             for (let sibling of parent.children) {
                 if (sibling.isInternal) {
-                    isOpen ? this.close(sibling.id) : this.open(sibling.id);
+                    if (isOpen)
+                        this.close(sibling.id, false);
+                    else
+                        this.open(sibling.id, false);
                 }
             }
+            this.redrawList();
             this.scrollTo(this.focusedNode);
         }
     }
     openAll() {
         utils.walk(this.root, (node) => {
             if (node.isInternal)
-                node.open();
+                this.open(node.id, false);
         });
+        this.redrawList();
     }
     closeAll() {
         utils.walk(this.root, (node) => {
             if (node.isInternal)
-                node.close();
+                this.close(node.id, false);
         });
+        this.redrawList();
     }
     /* Scrolling */
     scrollTo(identity, align = "smart") {

package/dist/module/interfaces/tree-api.test.js

@@ -10,3 +10,34 @@ test("tree.canDrop()", () => {
     expect(setupApi({ disableDrop: () => false }).canDrop()).toBe(true);
     expect(setupApi({ disableDrop: false }).canDrop()).toBe(true);
 });
+const rowData = [{ id: "a" }, { id: "b" }, { id: "c" }];
+test("rowHeight defaults to 24", () => {
+    const api = setupApi({});
+    expect(api.rowHeight).toBe(24);
+    expect(api.rowHeightAt(0)).toBe(24);
+});
+test("fixed numeric rowHeight", () => {
+    const api = setupApi({ data: rowData, rowHeight: 30 });
+    expect(api.rowHeight).toBe(30);
+    expect(api.rowHeightAt(0)).toBe(30);
+    expect(api.rowTopPosition(0)).toBe(0);
+    expect(api.rowTopPosition(2)).toBe(60);
+    expect(api.rowTopPosition(3)).toBe(90); // total list height
+});
+test("variable rowHeight function", () => {
+    const heights = { a: 10, b: 20, c: 40 };
+    const api = setupApi({
+        data: rowData,
+        rowHeight: (node) => heights[node.id],
+    });
+    // The back-compat getter falls back to the default for variable heights.
+    expect(api.rowHeight).toBe(24);
+    expect(api.rowHeightAt(0)).toBe(10);
+    expect(api.rowHeightAt(1)).toBe(20);
+    expect(api.rowTopPosition(0)).toBe(0);
+    expect(api.rowTopPosition(1)).toBe(10);
+    expect(api.rowTopPosition(2)).toBe(30);
+    expect(api.rowTopPosition(3)).toBe(70); // total list height
+    // Out-of-range index falls back to the default height, never an invalid 0.
+    expect(api.rowHeightAt(99)).toBe(24);
+});

package/dist/module/state/drag-slice.js

@@ -7,8 +7,7 @@ export function reducer(state = initialState().nodes.drag, action) {
         case "DND_DRAG_END":
             return Object.assign(Object.assign({}, state), { id: null, destinationParentId: null, destinationIndex: null, selectedIds: [] });
         case "DND_HOVERING":
-            if (action.parentId !== state.destinationParentId ||
-                action.index != state.destinationIndex) {
+            if (action.parentId !== state.destinationParentId || action.index != state.destinationIndex) {
                 return Object.assign(Object.assign({}, state), { destinationParentId: action.parentId, destinationIndex: action.index });
             }
             else {

package/dist/module/types/state.d.ts

@@ -1,2 +1,2 @@
 import { NodeApi } from "../interfaces/node-api";
-export type NodeState = typeof NodeApi.prototype["state"];
+export type NodeState = (typeof NodeApi.prototype)["state"];