Side navigation
#4886 closed enhancement (patchwelcome)
Opened July 11, 2009 07:23PM UTC
Closed April 17, 2011 12:27AM UTC
Last modified August 23, 2011 03:01PM UTC
Allow selective .die()
| Reported by: | ligreman | Owned by: | brandon |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | event | Version: | 1.3.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
If you bind multiple events to a child element (e.g. "p span") then use the die even to remove one of them it removes all events bound to that, instead of only one.
Example attached.
Attachments (1)
Change History (5)
Changed November 18, 2010 02:59AM UTC by comment:1
| milestone: | 1.4 |
|---|---|
| status: | new → open |
Changed November 19, 2010 02:47PM UTC by comment:2
| keywords: | die, event → needsdocs |
|---|---|
| priority: | major → low |
| summary: | Bug in event DIE. → Allow selective .die() |
| type: | bug → enhancement |
I'm recasting this as an enhancement, we need to document that (at least up to 1.4.4), the selector in .die() must exactly match the one in .live().
See the discussion here, with a proposed patch:
Changed April 17, 2011 12:27AM UTC by comment:3
| resolution: | → patchwelcome |
|---|---|
| status: | open → closed |
I'm fairly certain we have this documented now, but leaving the needsdocs just in case.
But yeah, I'm OK with the current functionality. If someone comes up with a smaller patch (than the one that was shown in the forum thread) and with a test case, please feel free to re-open!
Changed May 20, 2011 04:40PM UTC by comment:4
| keywords: | needsdocs |
|---|
Changed August 23, 2011 03:01PM UTC by comment:5
Test case example: http://jsfiddle.net/pPuG5/
http://jsfiddle.net/dmethvin/b8GqD/
Confirmed.