@gtkx/react 0.14.0 → 0.16.0

package/dist/nodes/toolbar-child.js

@@ -1,4 +1,4 @@
-import { batch, isObjectEqual } from "@gtkx/ffi";
+import { isObjectEqual } from "@gtkx/ffi";
 import { registerNodeClass } from "../registry.js";
 import { SlotNode } from "./slot.js";
 class ToolbarChildNode extends SlotNode {
@@ -17,23 +17,21 @@ class ToolbarChildNode extends SlotNode {
     }
     onChildChange(oldChild) {
         const toolbar = this.getToolbar();
-        batch(() => {
-            if (oldChild) {
-                const parent = oldChild.getParent();
-                if (parent && isObjectEqual(parent, toolbar)) {
-                    toolbar.remove(oldChild);
-                }
+        if (oldChild) {
+            const parent = oldChild.getParent();
+            if (parent && isObjectEqual(parent, toolbar)) {
+                toolbar.remove(oldChild);
             }
-            if (this.child) {
-                const position = this.getPosition();
-                if (position === "top") {
-                    toolbar.addTopBar(this.child);
-                }
-                else {
-                    toolbar.addBottomBar(this.child);
-                }
+        }
+        if (this.child) {
+            const position = this.getPosition();
+            if (position === "top") {
+                toolbar.addTopBar(this.child);
+            }
+            else {
+                toolbar.addBottomBar(this.child);
             }
-        });
+        }
     }
 }
 registerNodeClass(ToolbarChildNode);

package/dist/nodes/tree-list-item.d.ts

@@ -1,16 +1,18 @@
 import type { TreeListItemProps } from "../jsx.js";
 import type { Node } from "../node.js";
-import type { TreeStore } from "./internal/tree-store.js";
+import type { TreeItemData, TreeStore } from "./internal/tree-store.js";
 import { VirtualNode } from "./virtual.js";
 type Props = Partial<TreeListItemProps>;
+export declare const createTreeItemData: (props: Props) => TreeItemData;
 export declare class TreeListItemNode extends VirtualNode<Props> {
     static priority: number;
-    private store?;
-    private parentItemId?;
+    private store;
+    private parentItemId;
     private childNodes;
     static matches(type: string): boolean;
-    setStore(store?: TreeStore | null): void;
-    setParentItemId(parentId?: string | null): void;
+    setStore(store: TreeStore | null): void;
+    getChildNodes(): readonly TreeListItemNode[];
+    setParentItemId(parentId: string | null): void;
     getParentItemId(): string | null;
     appendChild(child: Node): void;
     insertBefore(child: Node, before: Node): void;

package/dist/nodes/tree-list-item.js

@@ -1,10 +1,15 @@
 import { registerNodeClass } from "../registry.js";
-import { scheduleAfterCommit } from "../scheduler.js";
 import { VirtualNode } from "./virtual.js";
+export const createTreeItemData = (props) => ({
+    value: props.value,
+    indentForDepth: props.indentForDepth,
+    indentForIcon: props.indentForIcon,
+    hideExpander: props.hideExpander,
+});
 export class TreeListItemNode extends VirtualNode {
     static priority = 1;
-    store;
-    parentItemId;
+    store = null;
+    parentItemId = null;
     childNodes = [];
     static matches(type) {
         return type === "TreeListItem";
@@ -15,36 +20,32 @@ export class TreeListItemNode extends VirtualNode {
             child.setStore(store);
         }
     }
+    getChildNodes() {
+        return this.childNodes;
+    }
     setParentItemId(parentId) {
         this.parentItemId = parentId;
     }
     getParentItemId() {
-        return this.parentItemId ?? null;
+        return this.parentItemId;
     }
     appendChild(child) {
         if (!(child instanceof TreeListItemNode)) {
             return;
         }
         child.setStore(this.store);
-        child.setParentItemId(this.props.id);
+        child.setParentItemId(this.props.id ?? null);
         this.childNodes.push(child);
-        scheduleAfterCommit(() => {
-            if (this.store && child.props.id !== undefined) {
-                this.store.addItem(child.props.id, {
-                    value: child.props.value,
-                    indentForDepth: child.props.indentForDepth,
-                    indentForIcon: child.props.indentForIcon,
-                    hideExpander: child.props.hideExpander,
-                }, this.props.id);
-            }
-        });
+        if (this.store && child.props.id !== undefined) {
+            this.store.addItem(child.props.id, createTreeItemData(child.props), this.props.id);
+        }
     }
     insertBefore(child, before) {
         if (!(child instanceof TreeListItemNode) || !(before instanceof TreeListItemNode)) {
             return;
         }
         child.setStore(this.store);
-        child.setParentItemId(this.props.id);
+        child.setParentItemId(this.props.id ?? null);
         const beforeIndex = this.childNodes.indexOf(before);
         if (beforeIndex === -1) {
             this.childNodes.push(child);
@@ -52,16 +53,9 @@ export class TreeListItemNode extends VirtualNode {
         else {
             this.childNodes.splice(beforeIndex, 0, child);
         }
-        scheduleAfterCommit(() => {
-            if (this.store && child.props.id !== undefined && before.props.id !== undefined) {
-                this.store.insertItemBefore(child.props.id, before.props.id, {
-                    value: child.props.value,
-                    indentForDepth: child.props.indentForDepth,
-                    indentForIcon: child.props.indentForIcon,
-                    hideExpander: child.props.hideExpander,
-                }, this.props.id);
-            }
-        });
+        if (this.store && child.props.id !== undefined && before.props.id !== undefined) {
+            this.store.insertItemBefore(child.props.id, before.props.id, createTreeItemData(child.props), this.props.id);
+        }
     }
     removeChild(child) {
         if (!(child instanceof TreeListItemNode)) {
@@ -82,20 +76,14 @@ export class TreeListItemNode extends VirtualNode {
         if (!this.store) {
             return;
         }
-        if (!oldProps ||
+        const propsChanged = !oldProps ||
             oldProps.id !== newProps.id ||
             oldProps.value !== newProps.value ||
             oldProps.indentForDepth !== newProps.indentForDepth ||
             oldProps.indentForIcon !== newProps.indentForIcon ||
-            oldProps.hideExpander !== newProps.hideExpander) {
-            if (newProps.id !== undefined) {
-                this.store.updateItem(newProps.id, {
-                    value: newProps.value,
-                    indentForDepth: newProps.indentForDepth,
-                    indentForIcon: newProps.indentForIcon,
-                    hideExpander: newProps.hideExpander,
-                });
-            }
+            oldProps.hideExpander !== newProps.hideExpander;
+        if (propsChanged && newProps.id !== undefined) {
+            this.store.updateItem(newProps.id, createTreeItemData(newProps));
         }
     }
 }

package/dist/nodes/tree-list-view.js

@@ -25,15 +25,20 @@ class TreeListViewNode extends WidgetNode {
             selectionMode: props.selectionMode,
             selected: props.selected,
             onSelectionChanged: props.onSelectionChanged,
-        });
-        this.itemRenderer = new TreeListItemRenderer();
+        }, rootContainer);
+        this.itemRenderer = new TreeListItemRenderer(this.signalStore);
         this.itemRenderer.setStore(this.treeList.getStore());
+        this.treeList.getStore().setOnItemUpdated((id) => this.itemRenderer.rebindItem(id));
         this.container.setFactory(this.itemRenderer.getFactory());
     }
     mount() {
         super.mount();
         this.container.setModel(this.treeList.getSelectionModel());
     }
+    unmount() {
+        this.itemRenderer.dispose();
+        super.unmount();
+    }
     appendChild(child) {
         if (!(child instanceof TreeListItemNode)) {
             throw new Error(`Cannot append '${child.typeName}' to 'TreeListView': expected x.TreeListItem`);
@@ -54,10 +59,10 @@ class TreeListViewNode extends WidgetNode {
     }
     updateProps(oldProps, newProps) {
         if (!oldProps || oldProps.renderItem !== newProps.renderItem) {
-            this.itemRenderer.setRenderFn(newProps.renderItem);
+            this.itemRenderer.setRenderFn(newProps.renderItem ?? null);
         }
         if (!oldProps || oldProps.estimatedItemHeight !== newProps.estimatedItemHeight) {
-            this.itemRenderer.setEstimatedItemHeight(newProps.estimatedItemHeight);
+            this.itemRenderer.setEstimatedItemHeight(newProps.estimatedItemHeight ?? null);
         }
         const previousModel = this.treeList.getSelectionModel();
         this.treeList.updateProps(oldProps ? filterProps(oldProps, RENDERER_PROP_NAMES) : null, filterProps(newProps, RENDERER_PROP_NAMES));

package/dist/nodes/virtual.d.ts

@@ -5,7 +5,7 @@ export declare class VirtualNode<P = Props> extends Node<undefined, P> {
     static matches(_type: string): boolean;
     static createContainer(): void;
     props: P;
-    constructor(typeName: string, props: P | undefined, container: undefined, rootContainer?: Container);
+    constructor(typeName: string, props: P | undefined, container: undefined, rootContainer: Container);
     appendChild(_child: Node): void;
     removeChild(_child: Node): void;
     insertBefore(_child: Node, _before: Node): void;

package/dist/nodes/widget.d.ts

@@ -3,14 +3,8 @@ import { Node } from "../node.js";
 import type { Container, ContainerClass, Props } from "../types.js";
 export declare class WidgetNode<T extends Gtk.Widget = Gtk.Widget, P extends Props = Props> extends Node<T, P> {
     static priority: number;
-    private motionController?;
-    private clickController?;
-    private keyController?;
-    private scrollController?;
-    private dragSourceController?;
-    private dropTargetController?;
     static matches(_type: string, containerOrClass?: Container | ContainerClass | null): boolean;
-    static createContainer(props: Props, containerClass: typeof Gtk.Widget): Container | null;
+    static createContainer(props: Props, containerClass: typeof Gtk.Widget, _rootContainer?: Container): Container | null;
     appendChild(child: Node): void;
     removeChild(child: Node): void;
     insertBefore(child: Node, before: Node): void;
@@ -18,12 +12,8 @@ export declare class WidgetNode<T extends Gtk.Widget = Gtk.Widget, P extends Pro
     private insertBeforeInsertable;
     updateProps(oldProps: P | null, newProps: P): void;
     private updateSizeRequest;
-    private updateEventControllerProp;
-    private ensureDragSource;
-    private ensureDropTarget;
-    private updateNotifyHandler;
-    private propNameToSignalName;
-    private getProperty;
+    private updateGrabFocus;
+    private getPropertyDefaultValue;
     private setProperty;
     private detachChildFromParent;
     private attachChild;