Opened 12 years ago
Closed 12 years ago
#8994 closed bug (fixed)
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
Change History (4)
comment:1 Changed 12 years ago by
| Component: | unfiled → effects |
|---|---|
| Owner: | set to mathias |
| Priority: | undecided → low |
| Status: | new → pending |
comment:2 Changed 12 years ago by
| 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.
comment:3 Changed 12 years ago by
| Status: | new → assigned |
|---|
comment:4 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Landing pull request 352. Slightly improved defaultDisplay() and Throbber of Doom. Fixes #8994.
More Details:
Note: See
TracTickets for help on using
tickets.
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.