Ticket #2390: 2390_2.patch

_whatsnew.html

@@ -107 +107 @@
                         source logic for comments in IE.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2387">#2387</a>] Fixed JavaScript
                         with list commands when the editable document is selected with Ctrl-A.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2390">#2390</a>] Fixed the issue where 
+                        indent styles in JavaScript-generated &lt;p&gt; blocks are erased in IE.</li> 
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>

editor/_source/internals/fckxhtml_ie.js

@@ -29 +29 @@
 
 FCKXHtml._AppendAttributes = function( xmlNode, htmlNode, node, nodeName )
 {
-        var aAttributes = htmlNode.attributes ;
+        var aAttributes = htmlNode.attributes,
+                bHasStyle ;
 
         for ( var n = 0 ; n < aAttributes.length ; n++ )
         {
@@ -47 +48 @@
                         // attribute. It returns "null" for the nodeValue.
                         else if ( sAttName == 'style' )
                         {
-                                var data = FCKTools.ProtectFormStyles( htmlNode ) ;
-                                sAttValue = htmlNode.style.cssText.replace( FCKRegexLib.StyleProperties, FCKTools.ToLowerCase ) ;
-                                FCKTools.RestoreFormStyles( htmlNode, data ) ;
+                                // Just mark it to do it later in this function.
+                                bHasStyle = true ;
+                                continue ;
                         }
                         // There are two cases when the oAttribute.nodeValue must be used:
                         //              - for the "class" attribute
@@ -80 +81 @@
                         this._AppendAttribute( node, sAttName, sAttValue || oAttribute.nodeValue ) ;
                 }
         }
+
+        // IE loses the style attribute in JavaScript-created elements tags. (#2390) 
+        if ( bHasStyle || htmlNode.style.cssText.length > 0 )
+        {
+                var data = FCKTools.ProtectFormStyles( htmlNode ) ;
+                var sStyleValue = htmlNode.style.cssText.replace( FCKRegexLib.StyleProperties, FCKTools.ToLowerCase ) ;
+                FCKTools.RestoreFormStyles( htmlNode, data ) ;
+                this._AppendAttribute( node, 'style', sStyleValue ) ;
+        }
 }
 
 // On very rare cases, IE is loosing the "align" attribute for DIV. (right align and apply bulleted list)