Skip to main content

Bug Tracker

Side navigation

#4945 closed enhancement (duplicate)

Opened July 22, 2009 08:03PM UTC

Closed July 22, 2009 11:25PM UTC

Enhancing "closest" method for delegation

Reported by: mike.helgeson Owned by:
Priority: minor Milestone: 1.4
Component: core Version: 1.3.2
Keywords: delegation Cc:
Blocked by: Blocking:
Description

I propose adding a second argument to the "closest" method that will act as the end point for searching up the document tree. It can behave just like the "context" argument in the jQuery mother function. It will optimize the performance of a common pattern I have seen in my own code since the addition of this method. The new usage could be as follows:

$("table").bind("click",function( event ){

var $td = $( event.target ).closest("td", event.currentTarget );

});

Because I bound the handler to "table" I do not want to search any higher than that element for selector matches. Furthermore, if there was a selector match outside of the containing element, it could potentially and accidentally be matched.

I have attached the modified method. It is a very small change, and makes "closest" even more powerful for advanced developers.

Attachments (1)
Change History (2)

Changed July 22, 2009 09:24PM UTC by mike.helgeson comment:1

Changed July 22, 2009 11:25PM UTC by joern comment:2

resolution: → duplicate
status: newclosed