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

This commit is contained in:
Dave Methvin 2011-09-25 22:04:52 -04:00
parent b4120a7430
commit c7838c3607
2 changed files with 7 additions and 7 deletions

View File

@ -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 );

View File

@ -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 ) {