marko 5.33.7 → 5.33.9

package/dist/runtime/helpers/class-value.js

@@ -3,7 +3,7 @@
 module.exports = function classHelper(arg) {
   switch (typeof arg) {
     case "string":
-      return arg || null;
+      return arg || undefined;
     case "object":
       var result = "";
       var sep = "";
@@ -25,9 +25,6 @@ module.exports = function classHelper(arg) {
         }
       }
 
-      return result || null;
-
-    default:
-      return null;
+      return result || undefined;
   }
 };

package/dist/runtime/helpers/style-value.js

@@ -7,7 +7,7 @@ var changeCase = require("./_change-case");
  */
 module.exports = function styleHelper(style) {
   if (!style) {
-    return null;
+    return;
   }
 
   var type = typeof style;
@@ -38,7 +38,7 @@ module.exports = function styleHelper(style) {
       }
     }
 
-    return styles || null;
+    return styles || undefined;
   }
 
   return style;

package/dist/runtime/helpers/tags-compat/runtime-html.js

@@ -55,14 +55,17 @@ exports.p = function (tagsAPI) {
       const input = _.i;
       const tagsRenderer = _.r;
       const renderFn = createRenderFn(tagsRenderer);
+      const willRerender = componentDef._wrr;
       const $global = out.global;
       const streamData = $global.streamData = $global.streamData || {};
 
-      $global.serializedGlobals = $global.serializedGlobals || {};
-      $global.serializedGlobals.componentIdToScopeId = true;
-      $global.componentIdToScopeId = $global.componentIdToScopeId || {};
-      $global.componentIdToScopeId[component.id] = streamData.scopeId || 0;
-      out.bf(out._Y_, component, true);
+      if (willRerender) {
+        $global.serializedGlobals = $global.serializedGlobals || {};
+        $global.serializedGlobals.componentIdToScopeId = true;
+        $global.componentIdToScopeId = $global.componentIdToScopeId || {};
+        $global.componentIdToScopeId[component.id] = streamData.scopeId || 0;
+      }
+      out.bf(out._Y_, component, willRerender);
       renderFn(out.beginAsync(), input, {}, streamData);
       out.ef();
     },
@@ -72,8 +75,10 @@ exports.p = function (tagsAPI) {
 
 
 
+
     {
-      t: TagsCompatId
+      t: TagsCompatId,
+      i: true
     },
     {}
   );

package/dist/runtime/vdom/VElement.js

@@ -203,14 +203,6 @@ VElement.prototype = {
     vElementByDOMNode.set(el, this);
 
     return el;
-  },
-
-  ce_: function (name) {
-    // We don't care about the namespaces since the there
-    // is no chance that attributes with the same name will have
-    // different namespaces
-    var value = this.c__[name];
-    return value != null && value !== false;
   }
 };
 
@@ -267,9 +259,9 @@ function virtualizeElement(node, virtualizeChildNodes, ownerComponent) {
   return vdomEl;
 }
 
-VElement.cf_ = virtualizeElement;
+VElement.ce_ = virtualizeElement;
 
-VElement.cg_ = function (fromEl, vFromEl, toEl) {
+VElement.cf_ = function (fromEl, vFromEl, toEl) {
   var fromFlags = vFromEl.t_;
   var toFlags = toEl.t_;
   var attrs = toEl.c__;
@@ -303,13 +295,25 @@ VElement.cg_ = function (fromEl, vFromEl, toEl) {
 
   if (toFlags & FLAG_SIMPLE_ATTRS && fromFlags & FLAG_SIMPLE_ATTRS) {
     if (oldAttrs["class"] !== (attrValue = attrs["class"])) {
-      fromEl.className = attrValue;
+      if (attrValue) {
+        fromEl.className = attrValue;
+      } else {
+        fromEl.removeAttribute("class");
+      }
     }
     if (oldAttrs.id !== (attrValue = attrs.id)) {
-      fromEl.id = attrValue;
+      if (attrValue) {
+        fromEl.id = attrValue;
+      } else {
+        fromEl.removeAttribute("id");
+      }
     }
     if (oldAttrs.style !== (attrValue = attrs.style)) {
-      fromEl.style.cssText = attrValue;
+      if (attrValue) {
+        fromEl.style.cssText = attrValue;
+      } else {
+        fromEl.removeAttribute("style");
+      }
     }
     return;
   }

package/dist/runtime/vdom/VNode.js

@@ -3,10 +3,10 @@ function VNode() {}
 
 VNode.prototype = {
   bT_: function (finalChildCount, ownerComponent) {
-    this.ch_ = finalChildCount;
-    this.ci_ = 0;
+    this.cg_ = finalChildCount;
+    this.ch_ = 0;
     this.bZ_ = null;
-    this.cj_ = null;
+    this.ci_ = null;
     this.bW_ = null;
     this.bX_ = null;
     this._J_ = ownerComponent;
@@ -21,24 +21,24 @@ VNode.prototype = {
       // If the DocumentFragment node has a first child then we will return that.
       // Otherwise, the DocumentFragment node is not *really* the first child and
       // we need to skip to its next sibling
-      return nestedFirstChild || firstChild.ck_;
+      return nestedFirstChild || firstChild.cj_;
     }
 
     return firstChild;
   },
 
-  get ck_() {
+  get cj_() {
     var nextSibling = this.bX_;
 
     if (nextSibling) {
       if (nextSibling.bY_) {
         var firstChild = nextSibling.ay_;
-        return firstChild || nextSibling.ck_;
+        return firstChild || nextSibling.cj_;
       }
     } else {
       var parentNode = this.bW_;
       if (parentNode && parentNode.bY_) {
-        return parentNode.ck_;
+        return parentNode.cj_;
       }
     }
 
@@ -46,16 +46,16 @@ VNode.prototype = {
   },
 
   bM_: function (child) {
-    this.ci_++;
+    this.ch_++;
 
     if (this.ca_ === "textarea") {
-      if (child.cl_) {
-        this.cb_ += child.cm_;
+      if (child.ck_) {
+        this.cb_ += child.cl_;
       } else {
         throw TypeError();
       }
     } else {
-      var lastChild = this.cj_;
+      var lastChild = this.ci_;
 
       child.bW_ = this;
 
@@ -65,14 +65,14 @@ VNode.prototype = {
         this.bZ_ = child;
       }
 
-      this.cj_ = child;
+      this.ci_ = child;
     }
 
     return child;
   },
 
   cd_: function finishChild() {
-    if (this.ci_ === this.ch_ && this.bW_) {
+    if (this.ch_ === this.cg_ && this.bW_) {
       return this.bW_.cd_();
     } else {
       return this;

package/dist/runtime/vdom/VText.js

@@ -3,20 +3,20 @@ var VNode = require("./VNode");
 
 function VText(value, ownerComponent) {
   this.bT_(-1 /* no children */, ownerComponent);
-  this.cm_ = value;
+  this.cl_ = value;
 }
 
 VText.prototype = {
-  cl_: true,
+  ck_: true,
 
   bV_: 3,
 
   bl_: function (host) {
-    return (host.ownerDocument || host).createTextNode(this.cm_);
+    return (host.ownerDocument || host).createTextNode(this.cl_);
   },
 
   bN_: function () {
-    return new VText(this.cm_);
+    return new VText(this.cl_);
   }
 };
 

package/dist/runtime/vdom/hot-reload.js

@@ -44,7 +44,7 @@ exports.t = runtime.t = function (typeName) {
                 instance.P_(input, out);
               };
 
-              instance.cn_ = true;
+              instance.cm_ = true;
               instance.am_();
               instance._t_ = false;
 
@@ -78,7 +78,7 @@ registry._C_ = function (typeName, id) {
   if (instances) {
     instances.add(instance);
     instance.once("destroy", function () {
-      if (!instance.cn_) {
+      if (!instance.cm_) {
         instances.delete(instance);
       }
     });

package/dist/runtime/vdom/morphdom/fragment.js

@@ -80,8 +80,8 @@ function createFragmentNode(startNode, nextNode, parentNode) {
 
 function beginFragmentNode(startNode, parentNode) {
   var fragment = createFragmentNode(startNode, null, parentNode);
-  fragment.co_ = function (nextNode) {
-    fragment.co_ = null;
+  fragment.cn_ = function (nextNode) {
+    fragment.cn_ = null;
     insertBefore(
       fragment.endNode,
       nextNode,
@@ -92,4 +92,4 @@ function beginFragmentNode(startNode, parentNode) {
 }
 
 exports._h_ = createFragmentNode;
-exports.cp_ = beginFragmentNode;
+exports.co_ = beginFragmentNode;

package/dist/runtime/vdom/morphdom/helpers.js

@@ -37,6 +37,6 @@ function removeChild(node) {
 
 exports.bc_ = insertBefore;
 exports.bd_ = insertAfter;
-exports.ck_ = nextSibling;
+exports.cj_ = nextSibling;
 exports.ay_ = firstChild;
 exports.be_ = removeChild;

package/dist/runtime/vdom/morphdom/index.js

@@ -11,8 +11,8 @@ var KeySequence = require("../../components/KeySequence");
 var VElement = require("../vdom").bv_;
 var fragment = require("./fragment");
 var helpers = require("./helpers");
-var virtualizeElement = VElement.cf_;
-var morphAttrs = VElement.cg_;
+var virtualizeElement = VElement.ce_;
+var morphAttrs = VElement.cf_;
 var keysByDOMNode = domData._G_;
 var componentByDOMNode = domData._l_;
 var vElementByDOMNode = domData._H_;
@@ -20,11 +20,11 @@ var detachedByDOMNode = domData.aT_;
 
 var insertBefore = helpers.bc_;
 var insertAfter = helpers.bd_;
-var nextSibling = helpers.ck_;
+var nextSibling = helpers.cj_;
 var firstChild = helpers.ay_;
 var removeChild = helpers.be_;
 var createFragmentNode = fragment._h_;
-var beginFragmentNode = fragment.cp_;
+var beginFragmentNode = fragment.co_;
 
 var ELEMENT_NODE = 1;
 var TEXT_NODE = 3;
@@ -161,7 +161,7 @@ function morphdom(fromNode, toNode, host, componentsContext) {
     var fromComponent;
 
     outer: while (curToNodeChild) {
-      toNextSibling = curToNodeChild.ck_;
+      toNextSibling = curToNodeChild.cj_;
       curToNodeType = curToNodeChild.bV_;
       curToNodeKey = curToNodeChild.bU_;
 
@@ -587,7 +587,7 @@ function morphdom(fromNode, toNode, host, componentsContext) {
           {
             // Both nodes being compared are Text or Comment nodes
             isCompatible = true;
-            var curToNodeValue = curToNodeChild.cm_;
+            var curToNodeValue = curToNodeChild.cl_;
             var curFromNodeValue = curFromNodeChild.nodeValue;
             if (curFromNodeValue !== curToNodeValue) {
               if (
@@ -638,10 +638,10 @@ function morphdom(fromNode, toNode, host, componentsContext) {
     }
 
     // We have processed all of the "to nodes".
-    if (fromNode.co_) {
+    if (fromNode.cn_) {
       // If we are in an unfinished fragment, we have reached the end of the nodes
       // we were matching up and need to end the fragment
-      fromNode.co_(curFromNodeChild);
+      fromNode.cn_(curFromNodeChild);
     } else {
       // If curFromNodeChild is non-null then we still have some from nodes
       // left over that need to be removed

package/dist/runtime/vdom/vdom.js

@@ -19,7 +19,7 @@ function virtualizeChildNodes(node, vdomParent, ownerComponent) {
 function virtualize(node, ownerComponent) {
   switch (node.nodeType) {
     case 1:
-      return VElement.cf_(node, virtualizeChildNodes, ownerComponent);
+      return VElement.ce_(node, virtualizeChildNodes, ownerComponent);
     case 3:
       return new VText(node.nodeValue, ownerComponent);
     case 11:
@@ -78,5 +78,5 @@ exports.bv_ = VElement;
 exports.bx_ = VText;
 exports.by_ = VComponent;
 exports.bz_ = VFragment;
-exports.cf_ = virtualize;
+exports.ce_ = virtualize;
 exports.bA_ = virtualizeHTML;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "marko",
-  "version": "5.33.7",
+  "version": "5.33.9",
   "description": "UI Components + streaming, async, high performance, HTML templating for Node.js and the browser.",
   "keywords": [
     "front-end",
@@ -64,8 +64,8 @@
     "build": "babel ./src --out-dir ./dist --extensions .js --copy-files --config-file ../../babel.config.js --env-name=production"
   },
   "dependencies": {
-    "@marko/compiler": "^5.35.6",
-    "@marko/translator-default": "^5.32.7",
+    "@marko/compiler": "^5.35.7",
+    "@marko/translator-default": "^5.32.8",
     "app-module-path": "^2.2.0",
     "argly": "^1.2.0",
     "browser-refresh-client": "1.1.4",

package/src/runtime/helpers/class-value.js

@@ -3,7 +3,7 @@
 module.exports = function classHelper(arg) {
   switch (typeof arg) {
     case "string":
-      return arg || null;
+      return arg || undefined;
     case "object":
       var result = "";
       var sep = "";
@@ -25,9 +25,6 @@ module.exports = function classHelper(arg) {
         }
       }
 
-      return result || null;
-
-    default:
-      return null;
+      return result || undefined;
   }
 };

package/src/runtime/helpers/style-value.js

@@ -7,7 +7,7 @@ var changeCase = require("./_change-case");
  */
 module.exports = function styleHelper(style) {
   if (!style) {
-    return null;
+    return;
   }
 
   var type = typeof style;
@@ -38,7 +38,7 @@ module.exports = function styleHelper(style) {
       }
     }
 
-    return styles || null;
+    return styles || undefined;
   }
 
   return style;

package/src/runtime/helpers/tags-compat/runtime-html.js

@@ -55,14 +55,17 @@ exports.p = function (tagsAPI) {
       const input = _.i;
       const tagsRenderer = _.r;
       const renderFn = createRenderFn(tagsRenderer);
+      const willRerender = componentDef._wrr;
       const $global = out.global;
       const streamData = ($global.streamData = $global.streamData || {});
 
-      $global.serializedGlobals = $global.serializedGlobals || {};
-      $global.serializedGlobals.componentIdToScopeId = true;
-      $global.componentIdToScopeId = $global.componentIdToScopeId || {};
-      $global.componentIdToScopeId[component.id] = streamData.scopeId || 0;
-      out.bf(out.___assignedKey, component, true);
+      if (willRerender) {
+        $global.serializedGlobals = $global.serializedGlobals || {};
+        $global.serializedGlobals.componentIdToScopeId = true;
+        $global.componentIdToScopeId = $global.componentIdToScopeId || {};
+        $global.componentIdToScopeId[component.id] = streamData.scopeId || 0;
+      }
+      out.bf(out.___assignedKey, component, willRerender);
       renderFn(out.beginAsync(), input, {}, streamData);
       out.ef();
     },
@@ -70,10 +73,12 @@ exports.p = function (tagsAPI) {
     "MARKO_DEBUG"
       ? {
           t: TagsCompatId,
+          i: true,
           d: true,
         }
       : {
           t: TagsCompatId,
+          i: true,
         },
     {},
   );

package/src/runtime/vdom/VElement.js

@@ -204,14 +204,6 @@ VElement.prototype = {
 
     return el;
   },
-
-  ___hasAttribute: function (name) {
-    // We don't care about the namespaces since the there
-    // is no chance that attributes with the same name will have
-    // different namespaces
-    var value = this.___attributes[name];
-    return value != null && value !== false;
-  },
 };
 
 inherit(VElement, VNode);
@@ -303,13 +295,25 @@ VElement.___morphAttrs = function (fromEl, vFromEl, toEl) {
 
   if (toFlags & FLAG_SIMPLE_ATTRS && fromFlags & FLAG_SIMPLE_ATTRS) {
     if (oldAttrs["class"] !== (attrValue = attrs["class"])) {
-      fromEl.className = attrValue;
+      if (attrValue) {
+        fromEl.className = attrValue;
+      } else {
+        fromEl.removeAttribute("class");
+      }
     }
     if (oldAttrs.id !== (attrValue = attrs.id)) {
-      fromEl.id = attrValue;
+      if (attrValue) {
+        fromEl.id = attrValue;
+      } else {
+        fromEl.removeAttribute("id");
+      }
     }
     if (oldAttrs.style !== (attrValue = attrs.style)) {
-      fromEl.style.cssText = attrValue;
+      if (attrValue) {
+        fromEl.style.cssText = attrValue;
+      } else {
+        fromEl.removeAttribute("style");
+      }
     }
     return;
   }