Ticket #1633: 1633.patch

_whatsnew.html

@@ -51 +51 @@
                         which stopped working in FCKeditor 2.6.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2039">#2039</a>] Fixed the locking up issue
                         in the Find/Replace dialog.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1633">#1633</a>] External styles should no
+                        longer interfere with the appearance of the editor and floating panels now.</li>
         </ul>
         <h3>
                 Version 2.6</h3>

editor/_source/classes/fckeditingarea.js

@@ -97 +97 @@
                 var sOverrideError = '<script type="text/javascript" _fcktemp="true">window.onerror=function(){return true;};</script>' ;
 
                 oIFrame.frameBorder = 0 ;
-                oIFrame.width = oIFrame.height = '100%' ;
+                oIFrame.style.width = oIFrame.style.height = '100%' ;
 
                 if ( FCK_IS_CUSTOM_DOMAIN && FCKBrowserInfo.IsIE )
                 {

editor/_source/classes/fckpanel.js

@@ -66 +66 @@
         else
         {
                 var oIFrame = this._IFrame = this._Window.document.createElement('iframe') ;
+                FCKTools.ResetStyles( oIFrame );
                 oIFrame.src                                     = 'javascript:void(0)' ;
                 oIFrame.allowTransparency       = true ;
                 oIFrame.frameBorder                     = '0' ;
                 oIFrame.scrolling                       = 'no' ;
-                oIFrame.width = oIFrame.height = 0 ;
+                oIFrame.style.width = oIFrame.style.height = '0px' ;
                 FCKDomTools.SetElementStyles( oIFrame,
                         {
                                 position        : 'absolute',
@@ -279 +280 @@
                         {
                                 var iWidth = eMainNode.offsetWidth || eMainNode.firstChild.offsetWidth ;
                                 var iHeight = eMainNode.offsetHeight ;
-                                me._IFrame.width = iWidth ;
-                                me._IFrame.height = iHeight ;
+                                me._IFrame.style.width = iWidth + 'px' ;
+                                me._IFrame.style.height = iHeight + 'px' ;
 
                         }, 0 ) ;
 
@@ -305 +306 @@
 
                 // It is better to set the sizes to 0, otherwise Firefox would have
                 // rendering problems.
-                this._IFrame.width = this._IFrame.height = 0 ;
+                this._IFrame.style.width = this._IFrame.style.height = '0px' ;
 
                 this._IsOpened = false ;
 

editor/_source/internals/fck.js

@@ -1175 +1175 @@
                 FCK.Events.FireEvent( "OnFocus" ) ;
         }
 }
+
+/*
+ * #1633 : Protect the editor iframe from external styles.
+ * Notice that we can't use FCKTools.ResetStyles here since FCKTools isn't
+ * loaded yet.
+ */
+(function()
+{
+        var el = window.frameElement ;
+        var width = el.width ;
+        var height = el.height ;
+        /^\d+$/.test( width ) && ( width += 'px' ) ;
+        /^\d+$/.test( height ) && ( height += 'px' ) ;
+        var sy = el.style ;
+        sy.border = sy.padding = sy.margin = 0 ;
+        sy.backgroundColor = 'transparent';
+        sy.backgroundImage = 'none';
+        sy.width = width ;
+        sy.height = height ;
+})() ;

editor/_source/internals/fckdialog.js

@@ -75 +75 @@
                         } ) ;
         }
 
-        var resetStyles = function( element )
-        {
-                element.style.cssText = 'margin:0;' +
-                        'padding:0;' +
-                        'border:0;' +
-                        'background-color:transparent;' +
-                        'background-image:none;' ;
-        }
-
         return {
                 /**
                  * Opens a dialog window using the standard dialog template.
@@ -113 +104 @@
 
                         // Setup the IFRAME that will hold the dialog.
                         var dialog = topDocument.createElement( 'iframe' ) ;
-                        resetStyles( dialog ) ;
+                        FCKTools.ResetStyles( dialog ) ;
                         dialog.src = FCKConfig.BasePath + 'fckdialog.html' ;
 
                         // Dummy URL for testing whether the code in fckdialog.js alone leaks memory.
@@ -179 +170 @@
                 {
                         // Setup the DIV that will be used to cover.
                         cover = topDocument.createElement( 'div' ) ;
-                        resetStyles( cover ) ;
+                        FCKTools.ResetStyles( cover ) ;
                         FCKDomTools.SetElementStyles( cover,
                                 {
                                         'position' : 'absolute',
@@ -195 +186 @@
                         if ( FCKBrowserInfo.IsIE && !FCKBrowserInfo.IsIE7 )
                         {
                                 var iframe = topDocument.createElement( 'iframe' ) ;
-                                resetStyles( iframe ) ;
+                                FCKTools.ResetStyles( iframe ) ;
                                 iframe.hideFocus = true ;
                                 iframe.frameBorder = 0 ;
                                 iframe.src = FCKTools.GetVoidUrl() ;

editor/_source/internals/fcktools.js

@@ -738 +738 @@
 
         return "javascript: void(0);" ;         // All other browsers.
 }
+
+FCKTools.ResetStyles = function( element )
+{
+        element.style.cssText = 'margin:0;' +
+                'padding:0;' +
+                'border:0;' +
+                'background-color:transparent;' +
+                'background-image:none;' ;
+}