Bug Tracker

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:

Change History (4)

comment:1 Changed 12 years ago by Rick Waldron

Component: unfiledeffects
Owner: set to 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.

comment:2 Changed 12 years ago by mathias

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.

comment:3 Changed 12 years ago by Rick Waldron

Status: newassigned

comment:4 Changed 12 years ago by Mathias Bynens

Resolution: fixed
Status: assignedclosed

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

More Details:

Note: See TracTickets for help on using tickets.