Fix list-component.js: Use correct bricks API methods - clear_widgets(), add_widget() instead of non-existent refresh() and appendChild()

wwwroot/list-component.js

@@ -48,39 +48,46 @@ bricks.List = class extends bricks.VBox {
     }
     
     renderItems() {
-        // Clear existing children by resetting subwidgets array
+        // Clear existing children
-        this.subwidgets = [];
+        this.clear_widgets();
         
-        // Create new item containers and add to subwidgets
+        // Render each item using urlwidget to load the item template
-        this.subwidgets = this.items.map((item, index) => {
+        this.items.forEach((item, index) => {
-            return {
+            const itemContainer = new bricks.VBox({
-                widgettype: "VBox",
+                width: '100%',
-                options: {
+                height: this.itemHeight + 'px'
-                    width: "100%",
+            });
-                    height: this.itemHeight + "px"
+            
-                },
+            // Store item data on the container for reference
-                _listItemData: item,
+            itemContainer._listItemData = item;
-                _listItemIndex: index,
+            itemContainer._listItemIndex = index;
-                binds: [{
+            
-                    wid: "self",
+            // Create urlwidget binding to load the item template
-                    event: "on_parent",
+            if (!itemContainer.binds) {
-                    actiontype: "urlwidget",
+                itemContainer.binds = [];
-                    target: "self",
+            }
+            
+            itemContainer.binds.push({
+                wid: 'self',
+                event: 'on_parent',
+                actiontype: 'urlwidget',
+                target: 'self',
                 options: {
                     url: this.item_template_url,
                     params: {
                         item: item,
                         index: index,
-                            id: "item_" + index
+                        id: 'item_' + index
                     }
                 },
-                    mode: "replace"
+                mode: 'replace'
-                }]
-            };
             });
             
-        // In bricks framework, updating subwidgets should automatically trigger re-render
+            // Use correct bricks API to add widget
-        // No need for explicit refresh or invalidate calls
+            this.add_widget(itemContainer);
+        });
+        
+        // No need for refresh() call - add_widget should handle rendering
     }
     
     // Public method to reload data