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)

stopImmediatePropagation[5843].diff (1.6 KB) - added by flesler September 10, 2008 09:35PM UTC.

Uses the expando to modify the event objects, doesn't break old code.

Change History (6)

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

#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:	Review → Commit
status:	new → assigned

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

resolution:	→ fixed
status:	assigned → closed

Applied at [5892].

Side navigation

#3355 closed feature (fixed)