#3905: Editor causes unauthenticated content warnings over SSL in FF 3.5
-------------------------------+--------------------------------------------
 Reporter:  jonathanc          |       Owner:  tobiasz.cudnik
     Type:  Bug                |      Status:  assigned      
 Priority:  Normal             |   Milestone:  CKEditor 3.1  
Component:  General            |     Version:  SVN (CKEditor)
 Keywords:  Confirmed Review-  |  
-------------------------------+--------------------------------------------
Changes (by fredck):

  * keywords:  Confirmed Review? => Confirmed Review-
  * milestone:  CKEditor 3.0 => CKEditor 3.1


Comment:

 This approach would be good for several reasons. The most important thing
 is that we would not need to use a "bridge" to send the data to the iframe
 to be written, which makes the code much clearer.

 Some things to be considered in the patch:

  * The "CKEDITOR._[ 'cke_htmlToLoad_' + editor.name ]" trick is not
 anymore needed. We can pass "data" directly to the createIFrame function
 at line 507.

  * The "onLoad" variable is not needed. The function can be passed
 directly to the on() call, and it's enough to call e.removeListener() to
 remove it at line 245.

  * The isCustomDomain variable has been removed from line 220, but
 isCustomDomain() is called twice in the patch, so it makes sense leaving
 that line intact and simply used the variable.

 In any case, these changes are too risky to be done at this stage. We can
 work on it as soon as we release the 3.0.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/3905#comment:5>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to