Ticket #10375 (closed bug: fixed)
Do not include `type` in jQuery.event.props
| Reported by: | frankvmurphy@… | Owned by: | dmethvin |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.7 |
| Component: | event | Version: | 1.7b1 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description (last modified by rwaldron) (diff)
The inclusion of type in the jQuery.event.props list will clobber the type that is set in jQuery.Event when it is passed through jQuery.event.fix()
Original ticket:
jQueryUI slider works well with latest 1.6 Sliding with mouse malfunctions. Get "RangeError: Maximum call stack size exceeded. Line 4155" On Safari,Chrome,Firefox
http://jsfiddle.net/3evgS/ (No 1.7b1 yet, but this should malfunction with it)
Change History
comment:1 Changed 21 months ago by rwaldron
- Priority changed from undecided to blocker
- Status changed from new to open
- Component changed from unfiled to selector
- Milestone changed from None to 1.7
comment:3 Changed 21 months ago by gnarf
Seems that removing "type" from $.event.props will fix this: http://jsfiddle.net/3evgS/3/
comment:4 Changed 21 months ago by rwaldron
- Owner set to dmethvin
- Status changed from open to assigned
- Description modified (diff)
- Summary changed from Stacksize exceeded on slider mouseevent to Do not include `type` in jQuery.event.props
comment:5 Changed 21 months ago by Dave Methvin
- Status changed from assigned to closed
- Resolution set to fixed
Fix #10375. Don't put type in jQuery.event.props
It's already set in jQuery.Event, and copying it can clobber values set by the caller, e.g., the UI widget factory.
Changeset: 96a44a86d870f2634c82a4148d22d0a07044ba77
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Confirmed.