Opened 11 years ago

Closed 11 years ago

#2312 closed Bug (invalid)

ReplaceTextarea() broken on IE 6

Reported by: Michael Slusarz Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.6.1
Keywords: Pending Cc:

Description (last modified by Frederico Caldeira Knabben)

From http://bugs.horde.org/ticket/6958 :

After playing around with this for about 2 hours, I have determined that this is a (critical) bug in fckeditor; more specifically, ReplaceTextarea(). I don't see what you are seeing, but instead I see the fckeditor load, then the window blinks for a second, and then it *looks* like it is completely loaded. Just 2 problems:

1.) all text that was previous in the textarea is gone and

2.) clicking on heading/font, or clicking anywhere in the textarea, outputs JS warnings. Strangely enough - if I put an alert() in the middle of the fckeditor code, the delay it takes for me to close the alert box appears to be enough to allow the loading process to proceed correctly. I see the textarea text correctly populated into the fckeditor window and i get no javascript errors when I click on various stuff. So my guess is that something is trying to be placed in a DOM element too quickly on IE6 before that element is ready.

The code in question works in all other browsers. As stated above, the code works on IE 6 if I put in an alert() statement anywhere in the fckeditor code (I put in an "alert("HERE!");" statement right after the definition of FCKBrowserInfo.

This code fires onload:

oFCKeditor.ReplaceTextarea()

In the base content of the page we have this declaration:

var oFCKeditor = new FCKeditor('message');

The textarea definition looks like this:

   <textarea class="composebody" tabindex="8" name="message" id="message" rows="20" cols="80">Foo</textarea>

Attachments (1)

2312_TC.html (2.2 KB) - added by Frederico Caldeira Knabben 11 years ago.
TC (save in _samples/html)

Download all attachments as: .zip

Change History (7)

comment:1 Changed 11 years ago by Martin Kou

The ReplaceTextArea HTML sample in our code package (_samples/html/sample13.html) seems to work correctly in IE6, so I don't really know what's wrong on your side.

Could you provide a test page for us so we can take a look and investigate?

comment:2 Changed 11 years ago by Martin Kou

Keywords: Pending added

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Description: modified (diff)

Changed 11 years ago by Frederico Caldeira Knabben

Attachment: 2312_TC.html added

TC (save in _samples/html)

comment:4 Changed 11 years ago by Frederico Caldeira Knabben

Priority: HighNormal

I've appended a TC page which is a copy of sample02.html. Just copy it inside the _samples/html folder.

It works well for me with IE6.

comment:5 Changed 11 years ago by Michael Slusarz

I'll take a look at the test file when I get back. I did make a video capture of the error: http://bigworm.curecanti.org/Movies/fckeditor-error.avi

The error I get is: 'ownerDocument.defaultView' is null or not an object

comment:6 Changed 11 years ago by Michael Slusarz

Resolution: invalid
Status: newclosed

In our distribution, we compress all JS files using both YUI Compressor and jsmin (http://www.crockford.com/javascript/jsmin.html) since fckeditor files are not optimally compressed. For some reason, jsmin compression of fckeditorcode_ie.js breaks IE. Don't know if it some strange IE 6 bug or some JS code in fckeditor that is not standards compliant, but the issue reported in this bug is resolved if we don't compress that file with jsmin.

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