Fix issue knockout#57 - when registering event handlers through jQuer…

…y, also trigger event handlers through jQuery
hunterloftis · Apr 5, 2011 · 1cbe9c5 · 1cbe9c5
1 parent 0ea4695
commit 1cbe9c5
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 9 deletions.
diff --git a/build/output/knockout-latest.debug.js b/build/output/knockout-latest.debug.js
@@ -190,7 +190,9 @@ ko.utils = new (function () {
             if (!(element && element.nodeType))
                 throw new Error("element must be a DOM node when calling triggerEvent");
 
-            if (typeof document.createEvent == "function") {
+            if (typeof jQuery != "undefined") {
+                jQuery(element)['trigger'](eventType);
+            } else if (typeof document.createEvent == "function") {
                 if (typeof element.dispatchEvent == "function") {
                     var eventCategory = (eventType == "click" ? "MouseEvents" : "HTMLEvents"); // Might need to account for other event names at some point
                     var event = document.createEvent(eventCategory);

diff --git a/build/output/knockout-latest.js b/build/output/knockout-latest.js
diff --git a/src/utils.js b/src/utils.js
@@ -174,7 +174,9 @@ ko.utils = new (function () {
             if (!(element && element.nodeType))
                 throw new Error("element must be a DOM node when calling triggerEvent");
 
-            if (typeof document.createEvent == "function") {
+            if (typeof jQuery != "undefined") {
+                jQuery(element)['trigger'](eventType);
+            } else if (typeof document.createEvent == "function") {
                 if (typeof element.dispatchEvent == "function") {
                     var eventCategory = (eventType == "click" ? "MouseEvents" : "HTMLEvents"); // Might need to account for other event names at some point
                     var event = document.createEvent(eventCategory);