Ticket #2531: 2531.patch

_whatsnew.html

@@ -77 +77 @@
                         condition at registering the FCKeditorAPI object in multiple editor scenarios.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2525">#2525</a>] Fixed JavaScript
                         error in Google Chrome when StartupShowBlocks is set to true.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2531">#2531</a>] The ENTER
+                        key will properly scroll to the cursor position when breaking long paragraphs.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>

editor/_source/classes/fckenterkey.js

@@ -536 +536 @@
                 }
 
                 if ( FCKBrowserInfo.IsGeckoLike )
-                        FCKDomTools.ScrollIntoView( eNextBlock || eNewBlock, false ) ;
-
+                {
+                        if ( eNextBlock )
+                        {
+                                // If we have split the block, adds a temporary span at the
+                                // range position and scroll relatively to it.
+                                var tmpNode = this.Window.document.createElement( 'span' ) ;
+                                
+                                // We need some content for Safari.
+                                tmpNode.innerHTML = ' ';
+                                
+                                oRange.InsertNode( tmpNode ) ;
+                                FCKDomTools.ScrollIntoView( tmpNode, false ) ;
+                                oRange.DeleteContents() ;
+                        }
+                        else
+                        {
+                                // We may use the above scroll logic for the new block case
+                                // too, but it gives some weird result with Opera.
+                                FCKDomTools.ScrollIntoView( eNextBlock || eNewBlock, false ) ;
+                        }
+                }
+                
                 oRange.Select() ;
         }