Bug Tracker

Modify

Ticket #12518 (closed bug: fixed)

Opened 19 months ago

Last modified 18 months ago

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:
Blocking: Blocked by:

Description

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.

Change History

comment:1 Changed 19 months ago by dmethvin

  • Priority changed from undecided to high
  • Status changed from new to open
  • Component changed from unfiled to event
  • Milestone changed from None to 1.9

comment:2 Changed 18 months ago by dmethvin

  • Owner set to fracmak2
  • Status changed from open to assigned

comment:3 Changed 18 months ago by Merrifield, Jay

  • Status changed from assigned to closed
  • Resolution set to fixed

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

Changeset: 408e5e08c23dda11e6e7a7f8dc44c9e6fcbe870d

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.