native-document 1.0.17-1.8 → 1.0.17-1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "native-document",
-  "version": "1.0.171.8",
+  "version": "1.0.171.9",
   "description": "A reactive JavaScript framework that preserves native DOM simplicity without sacrificing modern features",
   "author": "AfroCodeur <https://github.com/afrocodeur>",
   "license": "MIT",

package/src/core/wrappers/AttributesWrapper.js

@@ -1,38 +1,9 @@
 import Validator from '../utils/validator';
-import NativeDocumentError from '../errors/NativeDocumentError';
 import {BOOL_ATTRIBUTES_NAME, BOOLEAN_ATTRIBUTES} from './constants.js';
 import {Observable} from '../data/Observable';
 
 
 
-export const handleClassAttributeItem  = (element, className, value) => {
-    if (value.__$isObservableChecker) {
-        let lastClass = value.val();
-        if (typeof lastClass === 'string') {
-            element.classes.toggle(lastClass, true);
-            value.subscribe((currentValue) => {
-                element.classes.remove(lastClass);
-                element.classes.toggle(currentValue, true);
-                lastClass = currentValue;
-            });
-            return;
-        }
-    }
-
-    if (value.__$Observable) {
-        element.classes.toggle(className, value.val());
-        value.subscribe((shouldAdd) => element.classes.toggle(className, shouldAdd));
-        return;
-    }
-
-    if (value.$hydrate) {
-        value.$hydrate(element, className);
-        return;
-    }
-
-    element.classes.toggle(className, value);
-};
-
 /**
  * Applies a reactive class map to an HTMLElement.
  * Each key is a CSS class name; each value is a boolean or ObservableItem<boolean>.
@@ -44,7 +15,31 @@ export const handleClassAttributeItem  = (element, className, value) => {
 export const bindClassAttribute = (element, data) => {
     for(const className in data) {
         const value = data[className];
-        handleClassAttributeItem(element, className, value);
+        if (value.__$isObservableChecker) {
+            let lastClass = value.val();
+            if (typeof lastClass === 'string') {
+                element.classes.toggle(lastClass, true);
+                value.subscribe((currentValue) => {
+                    element.classes.remove(lastClass);
+                    element.classes.toggle(currentValue, true);
+                    lastClass = currentValue;
+                });
+                return;
+            }
+        }
+
+        if (value.__$Observable) {
+            element.classes.toggle(className, value.val());
+            value.subscribe((shouldAdd) => element.classes.toggle(className, shouldAdd));
+            return;
+        }
+
+        if (value.$hydrate) {
+            value.$hydrate(element, className);
+            return;
+        }
+
+        element.classes.toggle(className, value);
     }
 };
 

package/src/core/wrappers/ElementCreator.js

@@ -90,15 +90,8 @@ export const ElementCreator = {
         }
         if(Array.isArray(children)) {
             for(let i = 0, length = children.length; i < length; i++) {
-                let child = children[i];
-                if(child == null) {
-                    continue;
-                }
-                if(!child.__$isNativeNode) {
-                    child = ElementCreator.getChild(children[i]);
-                    if(child) {
-                        parent.appendChild(child);
-                    }
+                const child = ElementCreator.getChild(children[i]);
+                if(child === null) {
                     continue;
                 }
                 parent.appendChild(child);

package/src/core/wrappers/NDElement.js

@@ -5,7 +5,7 @@ import DebugManager from '../utils/debug-manager.js';
 import attributesWrapper, {
     bindAttributeWithObservable,
     bindClassAttribute,
-    bindStyleAttribute, handleClassAttributeItem,
+    bindStyleAttribute,
 } from './AttributesWrapper';
 
 /**
@@ -434,17 +434,6 @@ NDElement.prototype.style = function(style) {
     return this;
 };
 
-NDElement.prototype.className = function(classes) {
-    this.$element.classes.add(classes);
-    return this;
-};
-
-NDElement.prototype.class = function(className, value) {
-    handleClassAttributeItem(this.$element, className, value);
-    return this;
-};
-
-
 /**
  * Extends the NDElement prototype with new methods available to all NDElement instances.
  * Use this to add global methods to all NDElements.

package/src/core/wrappers/NdPrototype.js

@@ -29,9 +29,9 @@ EVENTS.forEach(eventSourceName => {
     NDElement.prototype['on'+eventSourceName] = function(callback = null, options = null) {
         if(!this.$element[inlineHandler] && !options) {
             this.$element[inlineHandler] = callback;
-        } else {
-            this.$element.addEventListener(eventName, callback, options);
+            return this;
         }
+        this.$element.addEventListener(eventName, callback, options);
         return this;
     };
 });
@@ -160,6 +160,7 @@ const _preventStop = function(element, eventName, callback, options) {
     const inlineHandler = 'on' + eventName;
     if(!element[inlineHandler] && !options) {
         element[inlineHandler] = handler;
+        return;
     }
     element.addEventListener(eventName, handler, options);
     return this;