Side navigation
#8994 closed bug (fixed)
Opened April 28, 2011 07:05AM UTC
Closed May 20, 2011 03:25PM UTC
Fix IE Throbber of Doom in `defaultDisplay()`
| Reported by: | mathias | Owned by: | mathias |
|---|---|---|---|
| Priority: | low | Milestone: | 1.next |
| Component: | effects | Version: | git |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
See http://bugs.jquery.com/ticket/8985#comment:2 and https://github.com/jquery/jquery/commit/a76decc47660591bb807aecf56825019da8b39fa#L0R580
Patch recommended by @jdalton.
Pull request (with some other minor enhancements): https://github.com/jquery/jquery/pull/352
Attachments (0)
Change History (4)
Changed April 28, 2011 03:56PM UTC by comment:1
| component: | unfiled → effects |
|---|---|
| owner: | → mathias |
| priority: | undecided → low |
| status: | new → pending |
Changed May 02, 2011 11:34AM UTC by comment:2
| status: | pending → new |
|---|
Perhaps I’m Doing It Wrong™, but I’m getting seemingly unrelated errors when running the unit tests in older browsers.
- Safari 3.2.1: http://i.imgur.com/467wI.png
- IE7 and IE8: http://i.imgur.com/2HFhJ.png
That said, the code change doesn’t seem to introduce new bugs, in any browser.
Changed May 07, 2011 07:46PM UTC by comment:3
| status: | new → assigned |
|---|
Just to make sure... this passes in:
I only want to make absolutely certain because the piece of code in question exhibits grossly varying behaviour across different browsers and even browser versions.