Opened 10 years ago

Closed 10 years ago

#3465 closed Bug (fixed)

Dialog tabs are causing Firefox throbber to spin indefinitely.

Reported by: Martin Kou Owned by: Martin Kou
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: Oracle Review+ Cc: Senthil

Description

To reproduce:

  1. Open replacebyclass.html in Firefox 3.
  2. Open the link dialog.
  3. Click on any tab in the link dialog.
  4. Close it.
  5. Open the link dialog again.
  6. Click on any tab in the link dialog.
  7. Firefox's throbber icon spins and will not stop, indicating it's loading something.

Attachments (1)

3465.patch (3.0 KB) - added by Martin Kou 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by Martin Kou

Priority: NormalHigh

comment:2 Changed 10 years ago by Martin Kou

Status: newassigned

comment:3 Changed 10 years ago by Martin Kou

Ok... after quite a bit of investigations, I've found the problem to be actually related to the iframe used in the file upload widget in the link and image dialogs.

Changed 10 years ago by Martin Kou

Attachment: 3465.patch added

comment:4 Changed 10 years ago by Martin Kou

Keywords: Review? added

This is definitely a browser bug, should be related to this ticket: https://bugzilla.mozilla.org/show_bug.cgi?id=360704.

While the throbber was triggered by style changes to the dialog pages, the actual cause for the throbber care the frameDocument.$.open(), frameDocument.$.write() and frameDocument.$.close() calls in the dialogui plugin. Adding a 500ms delay before drawing the iframe content seems to prevent the throbber from appearing. Even shorter delays (e.g. 100ms) doesn't work, but then the delay doesn't seem to depend on any network event.

comment:5 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

That's weird... but it works.

comment:6 Changed 10 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [3481].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy