Ticket #2297: 2297.patch

_whatsnew.html

@@ -64 +64 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2149">#2149</a>] CSS urls
                         with querystring parameters were not being accepted for CSS values in the configuration
                         file (like EditorAreaCSS).</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2149">#2197</a>] The TAB
+                        key will now have the default browser behavior if TabSpaces=0. It will move the
+                        focus out of the editor (expect on Safari).</li>
         </ul>
         <p>
         </p>

editor/_source/classes/fckenterkey.js

@@ -43 +43 @@
         oKeystrokeHandler.SetKeystrokes( [
                 [ 13            , 'Enter' ],
                 [ SHIFT + 13, 'ShiftEnter' ],
-                [ 9             , 'Tab' ],
                 [ 8                     , 'Backspace' ],
-                [ CTRL + 8              , 'CtrlBackspace' ],
+                [ CTRL + 8      , 'CtrlBackspace' ],
                 [ 46            , 'Delete' ]
         ] ) ;
 
-        if ( tabSpaces > 0 )
+        this.TabText = '' ;
+
+        // Safari by default inserts 4 spaces on TAB, while others make the editor
+        // loose focus. So, we need to handle it here to not include those spaces.
+        if ( tabSpaces > 0 || FCKBrowserInfo.IsSafari )
         {
-                this.TabText = '' ;
-                while ( tabSpaces-- > 0 )
+                while ( tabSpaces-- )
                         this.TabText += '\xa0' ;
+
+                oKeystrokeHandler.SetKeystrokes( [ 9, 'Tab' ] );
         }
 
         oKeystrokeHandler.AttachToElement( targetWindow.document ) ;