Ticket #4886 (closed enhancement: patchwelcome)
Allow selective .die()
| Reported by: | ligreman | Owned by: | brandon |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | event | Version: | 1.3.2 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
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
-
prueba.html
(991 bytes) -
added by ligreman 4 years ago.
- Example showing the bug
Change History
Changed 4 years ago by ligreman
-
attachment
prueba.html
added
comment:1 Changed 3 years ago by dmethvin
- Status changed from new to open
- Milestone 1.4 deleted
http://jsfiddle.net/dmethvin/b8GqD/
Confirmed.
comment:2 Changed 3 years ago by dmethvin
- Keywords needsdocs added; die, event removed
- Priority changed from major to low
- Type changed from bug to enhancement
- Summary changed from Bug in event DIE. to Allow selective .die()
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:
comment:3 Changed 2 years ago by john
- Status changed from open to closed
- Resolution set to patchwelcome
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!
comment:5 Changed 21 months ago by anonymous
Test case example: http://jsfiddle.net/pPuG5/
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Example showing the bug