Modify ↓
Ticket #11925 (closed bug: fixed)
special.teardown should pass the handle as second argument
| Reported by: | Krinkle | Owned by: | |
|---|---|---|---|
| Priority: | undecided | Milestone: | None |
| Component: | unfiled | Version: | 1.7.2 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
The code in jQuery.event.special.beforeunload.teardown assumes this second argument, but it is not being passed.
This causes the eventHandle to stay long after the last event has been unbound. (e.g. off('beforeunload', fn) does not remove handle from the property).
Change History
comment:2 Changed 12 months ago by Timo Tijhof
- Status changed from new to closed
- Resolution set to fixed
Fix #11925, Pass eventHandle to special.teardown. Closes gh-831.
- Added unit test to confirm. The third assertion fails without the fix in ./src/event.js Changeset: 9bb3494ce99889e05a7333e38e4da9579ce6af8e
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.
https://github.com/jquery/jquery/pull/831