library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-11-23 02:54:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Pre-compute delegated event path and handlers to comply with http://www.w3.org/TR/DOM-Level-3-Events/#event-flow

Browse Source
This commit is contained in:
Dave Methvin 2011-07-31 11:53:15 -04:00 committed by timmywil
parent f3a9d30812
commit 03f99ff995
1 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
src/event.js
View File

@ -385,23 +385,17 @@ jQuery.event = {
var handlers = ((jQuery._data( this, "events" ) || {})[ event.type ] || []), var handlers = ((jQuery._data( this, "events" ) || {})[ event.type ] || []),
delegateCount = handlers.delegateCount, delegateCount = handlers.delegateCount,
args = Array.prototype.slice.call( arguments, 0 ), args = Array.prototype.slice.call( arguments, 0 ),
cur = event.target, handlerQueue = [],
i, selMatch, matches, handleObj, sel, matched, related; i, cur, selMatch, matches, handleObj, sel, matched, related;
// Copy the handler list, in case one of the existing handlers adds/removes handlers
handlers = handlers.slice(0);
// Use the fix-ed jQuery.Event rather than the (read-only) native event // Use the fix-ed jQuery.Event rather than the (read-only) native event
args[0] = event; args[0] = event;
// Run delegates first; they may want to stop propagation beneath us // Determine handlers that should run if there are delegated events
// Avoid disabled elements in IE (#6911) and non-left-click bubbling in Firefox (#3861) // Avoid disabled elements in IE (#6911) and non-left-click bubbling in Firefox (#3861)
if ( delegateCount && cur && !cur.disabled && !(event.button && event.type === "click") ) { if ( delegateCount && !event.target.disabled && !(event.button && event.type === "click") ) {
// Let handlers know this is delegated, and from where; also used by .one() for ( cur = event.target; cur != this; cur = cur.parentNode || this ) {
event.delegateTarget = this;
for ( ; cur && cur != this && !event.isPropagationStopped(); cur = cur.parentNode ) {
selMatch = {}; selMatch = {};
matches = []; matches = [];
for ( i = 0; i < delegateCount; i++ ) { for ( i = 0; i < delegateCount; i++ ) {
@ -429,15 +423,24 @@ jQuery.event = {
} }
} }
if ( matches.length ) { if ( matches.length ) {
dispatch( cur, event, matches, 0, args ); handlerQueue.push({ elem: cur, matches: matches });
} }
} }
delete event.delegateTarget;
} }
// Copy the remaining (bound) handlers in case they're changed
handlers = handlers.slice( delegateCount );
// Run delegates first; they may want to stop propagation beneath us
event.delegateTarget = this;
for ( i=0; i < handlerQueue.length && !event.isPropagationStopped(); i++ ) {
dispatch( handlerQueue[i].elem, event, handlerQueue[i].matches, args );
}
delete event.delegateTarget;
// Run non-delegated handlers for this level // Run non-delegated handlers for this level
if ( delegateCount < handlers.length ) { if ( handlers.length ) {
dispatch( this, event, handlers, delegateCount, args ); dispatch( this, event, handlers, args );
} }
return event.result; return event.result;
@ -535,11 +538,11 @@ jQuery.event = {
}; };
// Run jQuery handler functions; called from jQuery.event.handle // Run jQuery handler functions; called from jQuery.event.handle
function dispatch( target, event, handlers, start, args ) { function dispatch( target, event, handlers, args ) {
var run_all = !event.exclusive && !event.namespace; var run_all = !event.exclusive && !event.namespace;
event.currentTarget = target; event.currentTarget = target;
for ( var j = start, l = handlers.length; j < l && !event.isImmediatePropagationStopped(); j++ ) { for ( var j = 0, l = handlers.length; j < l && !event.isImmediatePropagationStopped(); j++ ) {
var handleObj = handlers[ j ]; var handleObj = handlers[ j ];
// Triggered event must either 1) be non-exclusive and have no namespace, or // Triggered event must either 1) be non-exclusive and have no namespace, or
@ -702,6 +705,7 @@ jQuery.each({
mouseleave: "mouseout" mouseleave: "mouseout"
}, function( orig, fix ) { }, function( orig, fix ) {
jQuery.event.special[ orig ] = { jQuery.event.special[ orig ] = {
//TODO: fix this break and add a useful test case
setup: function( data ) { setup: function( data ) {
jQuery.event.add( this, fix, data && data.selector ? delegate : withinElement, orig ); jQuery.event.add( this, fix, data && data.selector ? delegate : withinElement, orig );
}, },