Ticket #2323: 2323.patch

_whatsnew.html

@@ -54 +54 @@
                         has pressed the Source button.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2314">#2314</a>] Corrected
                         mixed up Chinese translations for the blockquote command.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2323">#2323</a>] Fixed the issue
+                        where the show blocks command loses the current selection from the view area when editing
+                        a long document.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>

editor/_source/commandclasses/fckshowblocks.js

@@ -44 +44 @@
         else
                 body.className += ' FCK__ShowBlocks' ;
 
+        if ( FCKBrowserInfo.IsIE )
+        {
+                try
+                {
+                        FCK.EditorDocument.selection.createRange().select() ;
+                }
+                catch ( e )
+                {}
+        }
+        else
+        {
+                var focus = FCK.EditorWindow.getSelection().focusNode ;
+                if ( focus.nodeType != 1 )
+                        focus = focus.parentNode ;
+                FCKDomTools.ScrollIntoView( focus, false ) ;
+        }
+
         FCK.Events.FireEvent( 'OnSelectionChange' ) ;
 }
 

editor/_source/internals/fckdomtools.js

@@ -1004 +1004 @@
 
                 // Scroll the window to the desired position, if not already visible.
                 var currentScroll = FCKTools.GetScrollPosition( window ).Y ;
-                if ( offset > 0 && offset > currentScroll )
+                if ( offset > 0 && ( offset > currentScroll || offset < currentScroll - windowHeight ) )
                         window.scrollTo( 0, offset ) ;
         },