Ticket #2247: 2247_2.patch

_whatsnew.html

@@ -44 +44 @@
                         (Kudos to asuter for proposing the patch) </li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2234">#2234</a>] Added the
                         ability to create, modify and remove Div containers.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2247">#2247</a>] The SHIFT+SPACE
+                        keystroke will now produce a &amp;nbsp; character.</li>
         </ul>
         <p>
                 Fixed Bugs:</p>

editor/_source/commandclasses/fck_othercommands.js

@@ -613 +613 @@
                 range.Select() ;
         }
 } ;
+
+// FCKRuleCommand
+var FCKNbsp = function()
+{
+        this.Name = 'Non Breaking Space' ;
+}
+
+FCKNbsp.prototype =
+{
+        Execute : function()
+        {
+                FCK.InsertHtml( ' ' ) ;
+        },
+
+        GetState : function()
+        {
+                return ( FCK.EditMode != FCK_EDITMODE_WYSIWYG ? FCK_TRISTATE_DISABLED : FCK_TRISTATE_OFF ) ;
+        }
+} ;

editor/_source/internals/fck_gecko.js

@@ -359 +359 @@
         // Save an undo snapshot first.
         FCKUndo.SaveUndoStep() ;
 
+        // Firefox cannot handle HTML code with   or  . (See #2248)
+        // Convert them all to \xa0 first.
         if ( FCKBrowserInfo.IsGecko )
-        {
-                // Using the following trick,   present at the beginning and at
-                // the end of the HTML are preserved (#2248).
-                html = '<span id="__fakeFCKRemove1__" style="display:none;">fakeFCKRemove</span>' + html + '<span id="__fakeFCKRemove2__" style="display:none;">fakeFCKRemove</span>' ;
-        }
+                html = html.replace( /(&nbsp;|&#160;)/g, '\xa0' ) ;
 
         // Insert the HTML code.
         doc.execCommand( 'inserthtml', false, html ) ;
 
-        if ( FCKBrowserInfo.IsGecko )
-        {
-                // Remove the fake nodes.
-                FCKDomTools.RemoveNode( doc.getElementById('__fakeFCKRemove1__') ) ;
-                FCKDomTools.RemoveNode( doc.getElementById('__fakeFCKRemove2__') ) ;
-        }
-
         this.Focus() ;
 
         // Save the caret position before calling document processor.

editor/_source/internals/fckcommands.js

@@ -80 +80 @@
                 case 'NewPage'          : oCommand = new FCKNewPageCommand() ; break ;
                 case 'PageBreak'        : oCommand = new FCKPageBreakCommand() ; break ;
                 case 'Rule'                     : oCommand = new FCKRuleCommand() ; break ;
+                case 'Nbsp'                     : oCommand = new FCKNbsp() ; break ;
 
                 case 'TextColor'        : oCommand = new FCKTextColorCommand('ForeColor') ; break ;
                 case 'BGColor'          : oCommand = new FCKTextColorCommand('BackColor') ; break ;

fckconfig.js

@@ -142 +142 @@
         [ CTRL + 73 /*I*/, 'Italic' ],
         [ CTRL + 85 /*U*/, 'Underline' ],
         [ CTRL + SHIFT + 83 /*S*/, 'Save' ],
-        [ CTRL + ALT + 13 /*ENTER*/, 'FitWindow' ]
+        [ CTRL + ALT + 13 /*ENTER*/, 'FitWindow' ],
+        [ SHIFT + 32 /*SPACE*/, 'Nbsp' ]
 ] ;
 
 FCKConfig.ContextMenu = ['Generic','Link','Anchor','Image','Flash','Select','Textarea','Checkbox','Radio','TextField','HiddenField','ImageButton','Button','BulletedList','NumberedList','Table','Form','DivContainer'] ;