Re: QSA, the problem with ":scope", and naming

On 2011-10-20 10:14, Sean Hogan wrote:
> The primary use-case for matchesSelector() has been event-delegation,
> and this is the same for matches(). More specifically, consider the
> following scenario:
>
> jQuery adds a new event registration method that uses event delegation
> to mimic the behavior of:
> $(elem).find("> div > .thinger").bind(eventType, fn);
> The new method is called proxybind(), and the equivalent of the above is:
> $(elem).proxybind("> div > .thinger", eventType, fn);
>
> The event handling for proxybind() would invoke matches("> div >
> .thinger", [elem]) on elements between the event target and elem to find
> matching elements.

It may not be too late to introduce that behaviour into matchesSelector, 
with a switch based on the presence or absence of the 
refNodes/refElement parameter.

As currently specified, calling the following doesn't and shouldn't 
prepend :scope.

   el.matchesSelector("div .foo");

This one also matches the prefixed implementations in browsers, since 
most haven't started supporting :scope yet, and I don't believe 
Mozilla's experimental implementation [1] has landed yet.

As currently specified, calling this:

   el.matchesSelector("div .foo", ref);

Also doesn't prepend :scope automatically, but in that case, the ref 
nodes do nothing useful.  Authors have to use :scope explicitly for them 
to be useful as in something like:

   el.matchesSelector(":scope div .foo", ref);

Or

   el.matchesSelector("div:scope .foo", ref);

One thing we could possibly do is define that if ref nodes are passed, 
and the selector doesn't explicitly use :scope, then effectively prepend 
":scope ".  This would be exactly the same behaviour as that discussed 
for .findAll();

That wouldn't break compatibility with anything, optimises for a common 
case and avoids introducing two separate match methods.

e.g.
el.matchesSelector("div .foo");             // No ref, no magic :scope

el.matchesSelector("div .foo", ref);        // Implied, magic :scope
el.matchesSelector("+.foo", ref);           // Implied, magic :scope

el.matchesSelector(":scope div .foo", ref); // Explicit, no magic :scope
el.matchesSelector("div:scope .foo", ref);  // Explicit, no magic :scope

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=648722

-- 
Lachlan Hunt - Opera Software
http://lachy.id.au/
http://www.opera.com/

Received on Thursday, 20 October 2011 09:42:51 UTC