Minor cleanups to code. Futile effort to get IE to pass the unit test.

diff --git a/src/event.js b/src/event.js
index 00b332c5b1ba6c95de3b139f8d9ed0151e6a7e0e..9ad8d69b7cf33d84a756ad25bb5d0c5eccb175dc 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -488,7 +488,8 @@ jQuery.event = {
                props: "button buttons clientX clientY fromElement layerX layerY offsetX offsetY pageX pageY screenX screenY toElement wheelDelta".split(" "),
                filter: function( event, original ) {
                        var eventDoc, doc, body,
-                               button = original.button;
+                               button = original.button,
+                               fromElement = original.fromElement;
 
                        // Calculate pageX/Y if missing and clientX/Y available
                        if ( event.pageX == null && original.clientX != null ) {
@@ -501,8 +502,8 @@ jQuery.event = {
                        }
 
                        // Add relatedTarget, if necessary
-                       if ( !event.relatedTarget && original.fromElement ) {
-                               event.relatedTarget = original.fromElement === event.target ? original.toElement : original.fromElement;
+                       if ( !event.relatedTarget && fromElement ) {
+                               event.relatedTarget = fromElement === event.target ? original.toElement : fromElement;
                        }
 
                        // Add which for click: 1 === left; 2 === middle; 3 === right
@@ -523,7 +524,7 @@ jQuery.event = {
                // Create a writable copy of the event object and normalize some properties
                var originalEvent = event,
                        propHook = jQuery.event.propHooks[ event.type ] || {},
-                       copy =  propHook.props? this.props.concat( propHook.props ) : this.props;
+                       copy =  propHook.props ? this.props.concat( propHook.props ) : this.props;
 
                event = jQuery.Event( originalEvent );
 

diff --git a/test/unit/event.js b/test/unit/event.js
index 319569ab926f4edf77a8eec7e9f6bebf04aac45a..1ca3e7d78464bd3d015876b9e7cf876e3daa4f60 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -2386,9 +2386,8 @@ test("delegated events quickIs", function() {
 test("propHooks extensions", function() {
        expect( 3 );
 
-       jQuery( "<a id='hook-fixture' href=''></a>" ).appendTo( "#qunit-fixture" );
-
-       var $fixture = jQuery( "#hook-fixture" );
+       // IE requires focusable elements to be visible, so append to body
+       var $fixture = jQuery( "<input type='text' id='hook-fixture' />" ).appendTo( "body" );
 
        // Ensure the property doesn't exist
        $fixture.bind( "focus", function( event ) {