#2381: Double Iframes for FCKeditor
---------------------+------------------------------------------------------
Reporter: jefkin | Owner:
Type: Bug | Status: new
Priority: Normal | Milestone:
Component: General | Version:
Keywords: |
---------------------+------------------------------------------------------
Possibly asked and answered.
Sorry I don't have time to download and verify bugs, and I'm not sure what
component this even belongs to.
We're using Drupal 5.6 with the 5.x-2.2-beta2 module of FCKeditor.
What I suspect is that the error is only visible with this combination.
What I know is the error is completely avoidable.
in fckeditor.js
in function FCKeditor.ReplaceTextarea()
Add the following code at the very beginning:
----
var oIFrame = document.getElementById(this.InstanceName + '___Frame');
if (!oIFrame)
{
----
Then add the following code at the very end:
----
}
----
This protective if-block will prevent any duplicates from be formed, since
the code inside that if-block creates the iframe with then name:
'''this.InstanceName + '___Frame''''.
Yes, it's probably the fault of some other library further up. Shouldn't
matter. Code should do the right thing, and duplicate editors is bad.
This won't appreciably affect speed of the component, and is a good thing.
If I'm reporting this the wrong way, sorry, if I'm reporting to to the
wrong place, double my-bad. But there you go.
Jeff
--
Ticket URL: <http://dev.fckeditor.net/ticket/2381>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
