marko 4.24.3 → 4.24.4

package/dist/runtime/components/event-delegation.js

@@ -74,16 +74,6 @@ function addDelegatedEventHandlerToDoc(eventType, doc) {
   var listeners = doc[listenersAttachedKey] = doc[listenersAttachedKey] || {};
   if (!listeners[eventType]) {
     body.addEventListener(eventType, listeners[eventType] = function (event) {
-      var propagationStopped = false;
-
-      // Monkey-patch to fix #97
-      var oldStopPropagation = event.stopPropagation;
-
-      event.stopPropagation = function () {
-        oldStopPropagation.call(event);
-        propagationStopped = true;
-      };
-
       var curNode = event.target;
       if (!curNode) {
         return;
@@ -106,15 +96,29 @@ function addDelegatedEventHandlerToDoc(eventType, doc) {
       // Attributes will have the following form:
       // on<event_type>("<target_method>|<component_id>")
 
-      do {
-        if (target = getEventFromEl(curNode, propName)) {
-          delegateEvent(curNode, propName, target, event);
+      if (event.bubbles) {
+        var propagationStopped = false;
+
+        // Monkey-patch to fix #97
+        var oldStopPropagation = event.stopPropagation;
 
-          if (propagationStopped) {
-            break;
+        event.stopPropagation = function () {
+          oldStopPropagation.call(event);
+          propagationStopped = true;
+        };
+
+        do {
+          if (target = getEventFromEl(curNode, propName)) {
+            delegateEvent(curNode, propName, target, event);
+
+            if (propagationStopped) {
+              break;
+            }
           }
-        }
-      } while ((curNode = curNode.parentNode) && curNode.getAttribute);
+        } while ((curNode = curNode.parentNode) && curNode.getAttribute);
+      } else if (target = getEventFromEl(curNode, propName)) {
+        delegateEvent(curNode, propName, target, event);
+      }
     }, true);
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "marko",
-    "version": "4.24.3",
+    "version": "4.24.4",
     "license": "MIT",
     "description": "UI Components + streaming, async, high performance, HTML templating for Node.js and the browser.",
     "dependencies": {

package/src/runtime/components/event-delegation.js

@@ -79,16 +79,6 @@ function addDelegatedEventHandlerToDoc(eventType, doc) {
     body.addEventListener(
       eventType,
       (listeners[eventType] = function(event) {
-        var propagationStopped = false;
-
-        // Monkey-patch to fix #97
-        var oldStopPropagation = event.stopPropagation;
-
-        event.stopPropagation = function() {
-          oldStopPropagation.call(event);
-          propagationStopped = true;
-        };
-
         var curNode = event.target;
         if (!curNode) {
           return;
@@ -111,15 +101,29 @@ function addDelegatedEventHandlerToDoc(eventType, doc) {
         // Attributes will have the following form:
         // on<event_type>("<target_method>|<component_id>")
 
-        do {
-          if ((target = getEventFromEl(curNode, propName))) {
-            delegateEvent(curNode, propName, target, event);
+        if (event.bubbles) {
+          var propagationStopped = false;
+
+          // Monkey-patch to fix #97
+          var oldStopPropagation = event.stopPropagation;
 
-            if (propagationStopped) {
-              break;
+          event.stopPropagation = function() {
+            oldStopPropagation.call(event);
+            propagationStopped = true;
+          };
+
+          do {
+            if ((target = getEventFromEl(curNode, propName))) {
+              delegateEvent(curNode, propName, target, event);
+
+              if (propagationStopped) {
+                break;
+              }
             }
-          }
-        } while ((curNode = curNode.parentNode) && curNode.getAttribute);
+          } while ((curNode = curNode.parentNode) && curNode.getAttribute);
+        } else if ((target = getEventFromEl(curNode, propName))) {
+          delegateEvent(curNode, propName, target, event);
+        }
       }),
       true
     );