Merge pull request #258 from HapticX/dev

fix rendering system

Author: Ethosa

src/happyx/spa/components.nim

@@ -100,24 +100,24 @@ template reRenderTmpl*() =
     var
       current = document.querySelector("[data-" & self.uniqCompId & "]")
       elements = newSeq[Element]()
-    for tag in compTmpData.children:
+    for tag in compTmpData.childNodes:
       if not current.isNil:
         elements.add(current)
         current = current.nextSibling.Element
     # echo elements.len
     for i in countdown(elements.len-1, 0, 1):
       let
         elem = elements[i]
-        tag = compTmpData.children[i]
+        tag = compTmpData.childNodes[i]
         parent = elem.parentElement
         idx = index(elem)
-        length = parent.children.len
+        length = parent.childNodes.len
       # echo idx, ", ", length
       elem.remove()
       if length-1 == idx:
         parent.appendChild(tag)
       else:
-        parent.insertBefore(tag, parent.children[idx])
+        parent.insertBefore(tag, parent.childNodes[idx])
     if activeElement.hasAttribute("id"):
       let actElem = document.getElementById(activeElement.id)
       if not actElem.isNil:

src/happyx/spa/tag.nim

@@ -344,8 +344,7 @@ proc addArgIter*(self: TagRef, arg: string) =
   ## 
   ## See also `addArg function #addArg,TagRef,string`_
   when defined(js):
-    if self.attributes.len == 0:
-      self.setAttribute(cstring(arg), "")
+    self.setAttribute(cstring(arg), "")
   else:
     if self.args.len == 0:
       self.args.add(arg)