Small refactor to Element and Container (#355)

src/components/Container/index.ts

@@ -282,13 +282,7 @@ class Container extends Element {
      * @param index - The index to move the element to.
      */
     move(element: Element, index: number) {
-        let idx = -1;
-        for (let i = 0; i < this.dom.childNodes.length; i++) {
-            if (this.dom.childNodes[i].ui === element) {
-                idx = i;
-                break;
-            }
-        }
+        const idx = Array.prototype.indexOf.call(this.dom.childNodes, element.dom);
 
         if (idx === -1) {
             this.appendBefore(element, this.dom.childNodes[index]);
@@ -579,7 +573,6 @@ class Container extends Element {
      * @param node.root - The root node of the dom structure.
      * @param node.children - The children of the root node.
      * @returns The recursively appended element node.
-     *
      */
     protected _buildDomNode(node: { [x: string]: any; root?: any; children?: any; }): Container {
         const keys = Object.keys(node);

src/components/Element/index.ts

@@ -892,10 +892,7 @@ class Element extends Events {
      * Gets / sets the width of the Element in pixels. Can also be an empty string to remove it.
      */
     set width(value: number | string) {
-        if (typeof value === 'number') {
-            value = String(value) + 'px';
-        }
-        this.style.width = value;
+        this.style.width = typeof value === 'number' ? `${value}px` : value;
     }
 
     get width(): number {
@@ -906,10 +903,7 @@ class Element extends Events {
      * Gets / sets the height of the Element in pixels. Can also be an empty string to remove it.
      */
     set height(value: number | string) {
-        if (typeof value === 'number') {
-            value = String(value) + 'px';
-        }
-        this.style.height = value;
+        this.style.height = typeof value === 'number' ? `${value}px` : value;
     }
 
     get height(): number {
@@ -1102,7 +1096,7 @@ class Element extends Events {
 
 // Declare an additional property on the base Node interface that references the owner Element
 declare global {
-    interface Node { // eslint-disable-line no-unused-vars
+    interface Node {
         ui: Element;
     }
 }