Ticket #1643: 1643_3.patch

_whatsnew.html

@@ -57 +57 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/339">#339</a>] [<a
                         target="_blank" href="http://dev.fckeditor.net/ticket/681">#681</a>] The SpellerPages
                         spell checker will now completely ignore the presence of HTML tags in the text.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1643">#1643</a>] Resolved
+                        several "strict warning" messages in Firefox when running FCKeditor.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a>

editor/_source/classes/fckcontextmenu.js

@@ -156 +156 @@
                         return false ;
                 }
         }
+        return true ;
 }
 
 function FCKContextMenu_AttachedElement_OnContextMenu( ev, fckContextMenu, el )

editor/_source/classes/fckdomrange.js

@@ -99 +99 @@
                         this._UpdateElementInfo() ;
                         return docFrag ;
                 }
+                return null ;
         },
 
         CheckIsCollapsed : function()
         {
                 if ( this._Range )
                         return this._Range.collapsed ;
+
+                return false ;
         },
 
         Collapse : function( toStart )

editor/_source/classes/fcktoolbarspecialcombo.js

@@ -32 +32 @@
 {
         this.SourceView                 = false ;
         this.ContextSensitive   = true ;
+        this.FieldWidth                 = null ;
+        this.PanelWidth                 = null ;
+        this.PanelMaxHeight             = null ;
         //this._LastValue                       = null ;
 }
 

editor/_source/commandclasses/fckjustifycommands.js

@@ -39 +39 @@
                         switch ( alignValue )
                         {
                                 case 'left' :
-                                        return classes[0] ;
+                                        return classes[0] || null ;
                                 case 'center' :
-                                        return classes[1] ;
+                                        return classes[1] || null ;
                                 case 'right' :
-                                        return classes[2] ;
+                                        return classes[2] || null ;
                                 case 'justify' :
-                                        return classes[3] ;
+                                        return classes[3] || null ;
                         }
                 }
                 return null ;

editor/_source/internals/fck.js

@@ -474 +474 @@
                         window.onresize() ;
         },
 
-        // For the FocusManager
-        HasFocus : false,
-
-
         // This collection is used by the browser specific implementations to tell
         // which named commands must be handled separately.
         RedirectNamedCommands : new Object(),

editor/_source/internals/fckstyles.js

@@ -88 +88 @@
                                 var style = styles[ styleName ] ;
                                 var state = style.CheckActive( path ) ;
 
-                                if ( style._LastState != state )
+                                if ( state != ( style._LastState || null ) )
                                 {
                                         style._LastState = state ;
 

editor/dtd/fck_xhtml10strict.js

@@ -23 +23 @@
  */
 FCK.DTD = (function()
 {
-    X = FCKTools.Merge ;
+    var X = FCKTools.Merge ;
 
     var H,I,J,K,C,L,M,A,B,D,E,G,N,F ;
     A = {ins:1, del:1, script:1} ;

editor/dtd/fck_xhtml10transitional.js

@@ -23 +23 @@
  */
 FCK.DTD = (function()
 {
-    X = FCKTools.Merge ;
+    var X = FCKTools.Merge ;
 
     var A,L,J,M,N,O,D,H,P,K,Q,F,G,C,B,E,I ;
     A = {isindex:1, fieldset:1} ;
@@ -137 +137 @@
             changed the map to avoid breaking the links on pieces, having
             "<b>this is a </b><a><b>link</b> test</a>", instead of
             "<b>this is a <a>link</a></b><a> test</a>".
-*/
- No newline at end of file
+*/