cleanup template code

src/ui/web/Template.js

@@ -13,7 +13,10 @@ function classNames(obj, value) {
         }
     }, "");
 }
-/*
+/**
+    Bindable template. Renders once, and allows bindings for given nodes. If you need
+    to change the structure on a condition, use a subtemplate (if)
+
     supports
         - event handlers (attribute fn value with name that starts with on)
         - one way binding of attributes (other attribute fn value)
@@ -140,6 +143,16 @@ export default class Template {
         const node = document.createElement(name);
         
         if (attributes) {
+            this._setNodeAttributes(node, attributes);
+        }
+        if (children) {
+            this._setNodeChildren(node, children);
+        }
+
+        return node;
+    }
+
+    _setNodeAttributes(node, attributes) {
         for(let [key, value] of Object.entries(attributes)) {
             const isFn = typeof value === "function";
             // binding for className as object of className => enabled
@@ -157,7 +170,7 @@ export default class Template {
         }
     }
 
-        if (children) {
+    _setNodeChildren(node, children) {
         if (!Array.isArray(children)) {
             children = [children];
         }
@@ -172,9 +185,6 @@ export default class Template {
         }
     }
     
-        return node;
-    }
-
     _addReplaceNodeBinding(fn, renderNode) {
         let prevValue = fn(this._value);
         let node = renderNode(null);
@@ -215,7 +225,7 @@ export default class Template {
 }
 
 for (const tag of TAG_NAMES) {
-    Template.prototype[tag] = function(...params) {
-        return this.el(tag, ... params);
+    Template.prototype[tag] = function(attributes, children) {
+        return this.el(tag, attributes, children);
     };
 }