Skip to main content

Bug Tracker

Side navigation

#12518 closed bug (fixed)

Opened September 12, 2012 01:48AM UTC

Closed October 20, 2012 06:04PM UTC

Don't use offsetWidth in jQuery.event.trigger()

Reported by: dmethvin Owned by: fracmak2
Priority: high Milestone: 1.9
Component: event Version: 1.8.0
Keywords: Cc:
Blocked by: Blocking:

It's currently used to determine whether it's safe to call .focus() on an element without an error in oldIE. However, it causes a reflow and can be a performance killer. Since a direct display:none check won't catch parent-hidden elements I think we'll need to go back to a try/catch for now, but it can be removed in 2.0.

Attachments (0)
Change History (3)

Changed September 12, 2012 01:49AM UTC by dmethvin comment:1

component: unfiledevent
milestone: None1.9
priority: undecidedhigh
status: newopen

Changed October 15, 2012 03:04PM UTC by dmethvin comment:2

owner: → fracmak2
status: openassigned

Changed October 20, 2012 06:04PM UTC by Merrifield, Jay comment:3

resolution: → fixed
status: assignedclosed

Fixes #12518, removes an offsetWidth on focus/blur events for an <IE9 bug that caused a performance hit. Closes gh-958

Changeset: 408e5e08c23dda11e6e7a7f8dc44c9e6fcbe870d