Don't run direct handlers if delegate did .stopPropagation().

This commit is contained in:
Dave Methvin 2011-10-21 10:56:16 -04:00
parent c4cc343c9c
commit 9fabe2028f
2 changed files with 17 additions and 1 deletions

View File

@ -459,7 +459,7 @@ jQuery.event = {
delete event.delegateTarget; delete event.delegateTarget;
// Run non-delegated handlers for this level // Run non-delegated handlers for this level
if ( handlers.length ) { if ( handlers.length && !event.isPropagationStopped() ) {
dispatch( this, event, handlers, args ); dispatch( this, event, handlers, args );
} }

View File

@ -2054,6 +2054,22 @@ test(".delegate()/.undelegate()", function() {
jQuery("#body").undelegate("#nothiddendiv div", "click"); jQuery("#body").undelegate("#nothiddendiv div", "click");
}); });
test("stopPropagation() stops directly-bound events on delegated target", function() {
expect(1);
var markup = jQuery( '<div><p><a href="#">target</a></p></div>' );
markup
.on( "click", function() {
ok( false, "directly-bound event on delegate target was called" );
})
.on( "click", "a", function( e ) {
e.stopPropagation();
ok( true, "delegated handler was called" );
})
.find("a").click().end()
.remove();
});
test("undelegate all bound events", function(){ test("undelegate all bound events", function(){
expect(1); expect(1);