Bug Tracker

Opened 8 years ago

Closed 6 years ago

#6386 closed enhancement (fixed)

support data argument for live events via "event.special.live.add"

Reported by: mike.helgeson Owned by: dmethvin
Priority: low Milestone: 1.7
Component: event Version: 1.4.4
Keywords: live, 1.7-discuss Cc:
Blocked by: Blocking:

Description (last modified by dmethvin)

I am attempting to support "live" events in a custom special event, and the data object passed in the "bind" method gets passed into my special event "setup" function. This allows accepting that argument as options. This does not work with the "live" method. The "data" property is correctly set in the "live" handler "event" object, but not passed into my special event "setup" function.

It is a very small change indeed. On line 2029 of jQuery 1.4.2,in the "jQuery.event.special.live.add" function, simply add the "handleObj.data" property as the fourth argument to "jQuery.event.add". Alternatively, this could be handled within the "jQuery.event.add" method.

As a side note. I think the "liveHandler" and "liveConvert" methods, which are currently private, should be moved to the "jQuery.event.special.live" object. This would allow plugins to leverage more of the "live" features for development. I would also be able to patch the previously mentioned issue within my special event plugin, but I cannot because "liveHandler" is private.

Change History (14)

comment:1 Changed 7 years ago by snover

Milestone: 1.4.3

Resetting milestone to future.

comment:2 Changed 7 years ago by snover

Milestone: 1.5
Priority: low
Status: newopen
Summary: "event.special.live.add" does not pass data argument to "event.add"support data argument for live events via "event.special.live.add"
Type: bugenhancement
Version: 1.4.21.4.4

If you provide a patch on GitHub, that will help to get this issue resolved more quickly.

comment:3 Changed 7 years ago by john

Milestone: 1.next

Let's look at this for 1.7.

comment:4 Changed 7 years ago by john

Keywords: 1.7-discuss added

Nominating ticket for 1.7 discussion.

comment:5 Changed 7 years ago by Rick Waldron

Description: modified (diff)

+1,

comment:6 Changed 7 years ago by jaubourg

+0,

comment:7 Changed 7 years ago by ajpiano

Description: modified (diff)

+1,

comment:8 Changed 7 years ago by timmywil

Description: modified (diff)

+1,

comment:9 Changed 7 years ago by dmethvin

Description: modified (diff)

+1, on allowing data in delegated events, but not on exposing liveHandler etc because I'd like to rework that.

comment:10 Changed 6 years ago by john

Description: modified (diff)

+1, Seems simple enough.

comment:11 Changed 6 years ago by scottgonzalez

+1, sounds like a bug

comment:12 Changed 6 years ago by jzaefferer

+0

comment:13 Changed 6 years ago by dmethvin

Description: modified (diff)
Milestone: 1.next1.7
Owner: set to dmethvin
Status: openassigned

comment:14 Changed 6 years ago by dmethvin

Resolution: fixed
Status: assignedclosed

In 1.7 the data argument is available via handleObj.data. Determine whether the event is delegated or not by looking at handleObj.selector which will be undefined if it is not delegated.

Note: See TracTickets for help on using tickets.