[fix] ref() should call on Node mounted

TechQuery · TechQuery · commit 067b11ef54e5 · 2024-11-09T16:41:06.000+08:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "dom-renderer",
-    "version": "2.4.2",
+    "version": "2.4.3",
     "license": "LGPL-3.0-or-later",
     "author": "shiy2008@gmail.com",
     "description": "A light-weight DOM Renderer supports Web components standard & TypeScript language",
diff --git a/source/dist/DOMRenderer.ts b/source/dist/DOMRenderer.ts
@@ -61,8 +61,6 @@ export class DOMRenderer {
             reusedVNode || new VNode({ tagName: vNode.tagName, node: vNode.node }),
             vNode
         );
-        if (node) vNode.ref?.(node);
-
         return node;
     }
 
@@ -120,6 +118,8 @@ export class DOMRenderer {
             for (const vNode of deletingGroup[selector]) this.deleteNode(vNode);
 
         this.commitChildren(node, newNodes as ChildNode[]);
+
+        for (const { ref, node } of newList) ref?.(node);
     }
 
     handleCustomEvent(node: EventTarget, event: string) {

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "dom-renderer",`
`3`		`- "version": "2.4.2",`
	`3`	`+ "version": "2.4.3",`
`4`	`4`	`"license": "LGPL-3.0-or-later",`
`5`	`5`	`"author": "[email protected]",`
`6`	`6`	`"description": "A light-weight DOM Renderer supports Web components standard & TypeScript language",`
Original file line number	Diff line number	Diff line change
`@@ -61,8 +61,6 @@ export class DOMRenderer {`
`61`	`61`	`reusedVNode \|\| new VNode({ tagName: vNode.tagName, node: vNode.node }),`
`62`	`62`	`vNode`
`63`	`63`	`);`
`64`		`- if (node) vNode.ref?.(node);`
`65`		`-`
`66`	`64`	`return node;`
`67`	`65`	`}`
`68`	`66`
`@@ -120,6 +118,8 @@ export class DOMRenderer {`
`120`	`118`	`for (const vNode of deletingGroup[selector]) this.deleteNode(vNode);`
`121`	`119`
`122`	`120`	`this.commitChildren(node, newNodes as ChildNode[]);`
	`121`	`+`
	`122`	`+ for (const { ref, node } of newList) ref?.(node);`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`handleCustomEvent(node: EventTarget, event: string) {`