Opened 15 years ago
Closed 15 years ago
#4455 closed Bug (fixed)
Unable to start editing when image inside document not loaded
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed Review+ | Cc: | diego@… |
Description
If there's large image inside the editing document, the wysiwyg mode will be totally freezed while the image is loading.
Not sure if we could try to initialize the editing document a little bit earlier than 'onload', namely the 'onDomReady' technique.
Attachments (2)
Change History (12)
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
| Milestone: | CKEditor 3.1 |
|---|---|
| Version: | → SVN (CKEditor) |
comment:3 Changed 15 years ago by
| Milestone: | → CKEditor 3.1 |
|---|
Let's investigate it. It's an important issue.
Changed 15 years ago by
| Attachment: | 4455.patch added |
|---|
comment:4 Changed 15 years ago by
| Keywords: | Confirmed Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
The patch include the patch of dependent ticket #3598.
comment:5 Changed 15 years ago by
| Keywords: | Review- added; Review? removed |
|---|
I think you were talking about #3593 instead. The comments there make this patch not useful anymore.
Please do not mix the patches. You may indicated that a patch form other ticket must be also applied, or even point that we need to fix another ticket first.
Now, considering that the activationScript is appended to the document after the closing <html>, is the onload event really needed? Can't we call the "temporary function" directly?
Changed 15 years ago by
| Attachment: | 4455_2.patch added |
|---|
comment:6 Changed 15 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Ticket Manual Test added at :
http://ckeditor.t/tt/4455/1.html.
comment:7 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:9 Changed 15 years ago by
| Cc: | diego@… added |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
Sometimes the following code is appended to the content of the editor:
<script id="cke_actscrpt" type="text/javascript">window.parent.CKEDITOR._["contentDomReadyeditor"]( window );</script>
In my case, I save the content to the database to be used in a different context than a browser so the script tag creates a problem; in any case it should not be part of the real (user) content.
comment:10 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Please don't reopen fixed ticket unless you're sure it's the regression that brought by the fixing, open new ticket instead.
Ticket Test added at : http://ckeditor.t/tt/4455/1.html.