Opened 12 years ago
Closed 12 years ago
#10375 closed bug (fixed)
Do not include `type` in jQuery.event.props
| Reported by: | Owned by: | dmethvin | |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.7 |
| Component: | event | Version: | 1.7b1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description (last modified by )
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 (5)
comment:1 Changed 12 years ago by
| Component: | unfiled → selector |
|---|---|
| Milestone: | None → 1.7 |
| Priority: | undecided → blocker |
| Status: | new → open |
comment:2 Changed 12 years ago by
| Component: | selector → event |
|---|
comment:3 Changed 12 years ago by
Seems that removing "type" from $.event.props will fix this: http://jsfiddle.net/3evgS/3/
comment:4 Changed 12 years ago by
| Description: | modified (diff) |
|---|---|
| Owner: | set to dmethvin |
| Status: | open → assigned |
| Summary: | Stacksize exceeded on slider mouseevent → Do not include `type` in jQuery.event.props |
comment:5 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
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
Confirmed. This seems to be rooted in Sizzle.