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

Support event delegation with relative selectors. Fixes #10762. Closes gh-860.

Browse Source
This commit is contained in:
Richard Gibson 2012-07-23 12:48:29 -04:00 committed by Timmy Willison
parent a08a18b80a
commit e761e0c6e0
2 changed files with 14 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/traversing.js
View File

@ -1,7 +1,7 @@
var runtil = /Until$/, var runtil = /Until$/,
rparentsprev = /^(?:parents|prev(?:Until|All))/, rparentsprev = /^(?:parents|prev(?:Until|All))/,
isSimple = /^.[^:#\[\.,]*$/, isSimple = /^.[^:#\[\.,]*$/,
POS = jQuery.expr.match.globalPOS, rneedsContext = jQuery.expr.match.needsContext,
// methods guaranteed to produce a unique set when starting from a unique set // methods guaranteed to produce a unique set when starting from a unique set
guaranteedUnique = { guaranteedUnique = {
children: true, children: true,
@ -72,9 +72,9 @@ jQuery.fn.extend({
is: function( selector ) { is: function( selector ) {
return !!selector && ( return !!selector && (
typeof selector === "string" ? typeof selector === "string" ?
// If this is a positional selector, check membership in the returned set // If this is a positional/relative selector, check membership in the returned set
// so $("p:first").is("p:last") won't return true for a doc with two "p". // so $("p:first").is("p:last") won't return true for a doc with two "p".
POS.test( selector ) ? rneedsContext.test( selector ) ?
jQuery( selector, this.context ).index( this[0] ) >= 0 : jQuery( selector, this.context ).index( this[0] ) >= 0 :
jQuery.filter( selector, this ).length > 0 : jQuery.filter( selector, this ).length > 0 :
this.filter( selector ).length > 0 ); this.filter( selector ).length > 0 );
@ -85,7 +85,7 @@ jQuery.fn.extend({
i = 0, i = 0,
l = this.length, l = this.length,
ret = [], ret = [],
pos = POS.test( selectors ) || typeof selectors !== "string" ? pos = rneedsContext.test( selectors ) || typeof selectors !== "string" ?
jQuery( selectors, context || this.context ) : jQuery( selectors, context || this.context ) :
0; 0;

17
test/unit/event.js
View File

@ -2342,17 +2342,20 @@ test("jQuery.off using dispatched jQuery.Event", function() {
.remove(); .remove();
}); });
test( "delegated event with delegateTarget-relative selector (#)", function() { test( "delegated event with delegateTarget-relative selector", function() {
expect(1); expect(2);
var markup = jQuery( '<ul><li><ul id="u1"><li id="f1"></li></ul></li>' ).appendTo("body"); var markup = jQuery( '<ul><li><a id="a0"></a><ul id="ul0"><li><a id="a0_0"></a></li><li><a id="a0_1"></a></li></ul></li></ul>' ).appendTo("body");
markup markup
.find("#u1") .on( "click", ">li>a", function() {
.on( "click", "li:first", function() { ok( this.id === "a0", "child li was clicked" );
ok( this.id === "f1" , "first li under #u1 was clicked" ); })
.find("#ul0")
.on( "click", "li:first>a", function() {
ok( this.id === "a0_0" , "first li under #u10 was clicked" );
}) })
.find("#f1").click().end()
.end() .end()
.find("a").click().end()
.remove(); .remove();
}); });