Bug Tracker

Opened 5 years ago

Closed 5 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 5 years ago by rwaldron

  • Component changed from unfiled to effects
  • Owner set to mathias
  • Priority changed from undecided to low
  • Status changed from new to pending

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 5 years ago by mathias

  • Status changed from pending to new

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 5 years ago by rwaldron

  • Status changed from new to assigned

comment:4 Changed 5 years ago by Mathias Bynens

  • Resolution set to fixed
  • Status changed from assigned to closed

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

More Details:

Note: See TracTickets for help on using tickets.