Side navigation
#11101 closed bug (fixed)
Opened December 24, 2011 04:42PM UTC
Closed June 11, 2012 10:10PM UTC
Deprecate "exclusive" events option from trigger method
| Reported by: | jzaefferer | Owned by: | dmethvin |
|---|---|---|---|
| Priority: | low | Milestone: | 1.8 |
| Component: | event | Version: | 1.7.1 |
| Keywords: | Cc: | dmethvin | |
| Blocked by: | Blocking: |
Description
Apparently trigger allows you to specify to trigger only non-namespaced events, via .trigger("click!"). Here's the implementation: https://github.com/jquery/jquery/blob/4534db196bf9475c79f74d6b62ebc866c27d06d9/src/event.js#L250-254 (line 250-254)
I don't see any mention of that in the documentation: http://api.jquery.com/trigger/
So either it should either be removed or documented.
Attachments (0)
Change History (5)
Changed December 26, 2011 01:52PM UTC by comment:1
| cc: | → DaveMethvin |
|---|---|
| component: | unfiled → effects |
| keywords: | → needsreview |
| priority: | undecided → low |
Changed December 26, 2011 02:57PM UTC by comment:2
| cc: | DaveMethvin → dmethvin |
|---|---|
| component: | effects → event |
| keywords: | needsreview |
| owner: | → dmethvin |
| status: | new → assigned |
We use exclusive events internally, in data events. However we've talked about deprecating those and when we remove them we can also remove exclusive events. That would be my preference.
These have never been in the formal documentation but have been mentioned in a few places:
http://www.learningjquery.com/2007/09/namespace-your-events
http://longgoldenears.blogspot.com/2010/02/jquery-namespaced-events-exclusive.html
Changed March 03, 2012 10:29PM UTC by comment:3
| milestone: | None → 1.8 |
|---|
Changed May 07, 2012 02:54PM UTC by comment:4
| summary: | Document or remove exclusive events option from trigger method → Deprecate "exclusive" events option from trigger method |
|---|
Changed June 11, 2012 10:10PM UTC by comment:5
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
I will mark this as done since there are no docs changes. This ticket will serve as notice. Removal will be done as a separate ticket in 1.9.
Adding needsreview. I personally think this should be documented, however if there's a good reason it hasn't been to date, it would be interesting to find out what it is :)