Bug Tracker

Ticket #10375 (closed bug: fixed)

Opened 3 years ago

Last modified 3 years ago

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 3 years 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

Confirmed.

Last edited 3 years ago by rwaldron (previous) (diff)

comment:2 Changed 3 years ago by rwaldron

  • Component changed from selector to event

comment:3 Changed 3 years ago by gnarf

Seems that removing "type" from $.event.props will fix this:  http://jsfiddle.net/3evgS/3/

comment:4 Changed 3 years 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 3 years 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

Note: See TracTickets for help on using tickets.