Skip to main content

Bug Tracker

Side navigation

#3355 closed feature (fixed)

Opened September 10, 2008 09:34PM UTC

Closed October 18, 2008 10:46PM UTC

Add stopImmediatePropagation to event objects

Reported by: flesler Owned by: flesler
Priority: minor Milestone: 1.3
Component: event Version: 1.2.6
Keywords: stopimmediatepropagation Cc: john, joern, brandon, davidserduke
Blocked by: Blocking:
Description

We got requests in the past, for a way to stop the execution of other handlers from within one.

I realize this could be easily done, by extending event objects with a method that is already a standard (DOM 3 events, included in AS3).

Check this and this for additional info.

Attachments (1)
Change History (6)

Changed September 10, 2008 09:46PM UTC by flesler comment:1

Related:

#2610, #1948, #399.

Changed September 11, 2008 07:39AM UTC by joern comment:2

Good concept that fixes an annoying issue.

The patch could be improved a bit by replacing the "event[expando] == -1" bit with something more clear. Use a value that makes it obvious that its related to event propagation, not a magic number like -1.

Changed September 11, 2008 12:58PM UTC by nathanhammon comment:3

Yes please. This is definitely something worth adding.

Changed September 11, 2008 02:35PM UTC by flesler comment:4

The expando shouldn't be a number higher than 0, or could be misused by jQuery.data.

Could be a string... but what for, it's just longer/heavier.

What would you propose Joern ?

Changed October 18, 2008 10:46PM UTC by flesler comment:5

need: ReviewCommit
status: newassigned

Changed October 18, 2008 10:46PM UTC by flesler comment:6

resolution: → fixed
status: assignedclosed

Applied at [5892].