Bug Tracker

Ticket #8994 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Fix IE Throbber of Doom in `defaultDisplay()`

Reported by: mathias Owned by: mathias
Priority: low Milestone: 1.next
Component: effects Version: git
Keywords: Cc:
Blocking: Blocked by:

Change History

comment:1 Changed 5 years ago by rwaldron

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

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

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

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

More Details:

Note: See TracTickets for help on using tickets.