Fixes Ticket #12518 #958

Closed
wants to merge 2 commits into
from

Projects

None yet

5 participants

@fracmak
fracmak commented Oct 15, 2012

removes an offsetWidth on focus/blur events for an <IE9 bug that caused a performance hit and replaces it with a try/catch block.

There was already a unit test in place to cover this.

This bug is tricky to trigger because you actually need an IE8 native browser in windows XP, not an IE9 in IE8 mode

Merrifield, Jay Fixes #12518, removes an offsetWidth on focus/blur events for an <IE9…
… bug that caused a performance hit
f4a7c75
@rwaldron
Member

reviewing

@rwaldron rwaldron closed this Oct 15, 2012
@rwaldron rwaldron reopened this Oct 15, 2012
@rwaldron
Member

Whoops! Sorry about that...

@rwaldron
Member
@mikesherov
Member

Cool. Looking now.

@mikesherov
Member

👍 LGTM

@dmethvin
Member

/botio test

@jquerybot
Collaborator

From: jQuery Bot.io


Received

Command cmd_test from @dmethvin received. Current queue size: 0

Live output at: http://swarm.jquery.org:8000/3daed2ba7944475/output.txt

@jquerybot
Collaborator

From: jQuery Bot.io


Failed

Full output at http://swarm.jquery.org:8000/3daed2ba7944475/output.txt

Total script time: 0.11 mins

@fracmak
fracmak commented Oct 15, 2012

merge with master, no conflicts, want to try again?

@mikesherov mikesherov added a commit that closed this pull request Oct 20, 2012
Merrifield, Jay Fixes #12518, removes an offsetWidth on focus/blur events for an <IE9…
… bug that caused a performance hit. Closes gh-958
408e5e0
@mescoda mescoda pushed a commit to mescoda/jquery that referenced this pull request Nov 4, 2014
Merrifield, Jay Fixes #12518, removes an offsetWidth on focus/blur events for an <IE9…
… bug that caused a performance hit. Closes gh-958
08824c7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment