Opened 17 years ago
Closed 17 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 )
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)
Change History (7)
comment:1 Changed 17 years ago by
comment:2 Changed 17 years ago by
Keywords: | Pending added |
---|
comment:3 Changed 17 years ago by
Description: | modified (diff) |
---|
comment:4 Changed 17 years ago by
Priority: | High → Normal |
---|
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 17 years ago by
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 17 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
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.
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?