Changeset 3825

Timestamp:

11/14/07 21:55:45 (9 months ago)

Author:

brandon.aaron

Message:

Fix for #1925 and removed some unnecessary browser sniffing

Files:

• trunk/jquery/src/event.js (modified) (5 diffs)

trunk/jquery/src/event.js

@@ -20 +20 @@
         // if data is passed, bind to handler 
         if( data != undefined ) { 
-                // Create temporary function pointer to original handler 
+            // Create temporary function pointer to original handler 
             var fn = handler; 
 
@@ -254 +254 @@
         
         // Fix target property, if necessary
-        if ( !event.target && event.srcElement )
-            event.target = event.srcElement;
+        if ( !event.target )
+            event.target = event.srcElement || document; // Fixes #1925 where srcElement might not be defined either
                 
         // check if target is a textnode (safari)
-        if (jQuery.browser.safari && event.target.nodeType == 3)
+        if ( event.target.nodeType == 3 )
             event.target = originalEvent.target.parentNode;
 
@@ -402 +402 @@
             }
             // Remove event listener to avoid memory leak
-            if ( jQuery.browser.mozilla || jQuery.browser.opera )
+            if ( document.removeEventListener )
                 document.removeEventListener( "DOMContentLoaded", jQuery.ready, false );
         }
@@ -425 +425 @@
     readyBound = true;
 
-    // If Mozilla is used
-    if ( jQuery.browser.mozilla || jQuery.browser.opera )
+    // Mozilla, Opera and webkit nightlies currently support this event
+    if ( document.addEventListener )
         // Use the handy event callback
         document.addEventListener( "DOMContentLoaded", jQuery.ready, false );
@@ -432 +432 @@
     // If Safari or IE is used
     // Continually check to see if the document is ready
-    else (function(){
+    if  (jQuery.browser.msie || jQuery.browser.safari ) (function(){
         try {
             // If IE is used, use the trick by Diego Perini