Skip to main content

Bug Tracker

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:
Attachments (0)
Change History (4)

Changed April 28, 2011 03:56PM UTC by rwaldron comment:1

component: unfiledeffects
owner: → mathias
priority: undecidedlow
status: newpending

Just to make sure... this passes in:

  • FF 3.0.x, 3.6.x, 4.x
  • Chrome current-1 ( 10, 11 )
  • Safari 3, 4, 5
  • IE 6, 7, 8, 9
  • Opera current-1 ( 11, 11.01 )

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.

Changed May 02, 2011 11:34AM UTC by mathias comment:2

status: pendingnew

Perhaps I’m Doing It Wrong™, but I’m getting seemingly unrelated errors when running the unit tests in older browsers.

That said, the code change doesn’t seem to introduce new bugs, in any browser.

Changed May 07, 2011 07:46PM UTC by rwaldron comment:3

status: newassigned

Changed May 20, 2011 03:25PM UTC by Mathias Bynens comment:4

resolution: → fixed
status: assignedclosed

Landing pull request 352. Slightly improved defaultDisplay() and Throbber of Doom. Fixes #8994.

More Details:

Changeset: b43910a496ccb2b44dcc11b64667e21521aa4994