Ticket #2320: 2320.patch

_whatsnew.html

@@ -64 +64 @@
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2322">#2322</a>] Fixed the issue
                         where the fit window command loses the current selection and scroll position in the
                         editing area.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2320">#2320</a>] Fixed the issue
+                        where the Find/Replace dialog scrolls the entire page.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>

editor/_source/internals/fckdomtools.js

@@ -989 +989 @@
                 // Appends the height it we are about to align the bottoms.
                 if ( alignTop === false )
                 {
-                        offset += element.offsetHeight ;
+                        offset += element.offsetHeight || 0 ;
 
                         // Consider the margin in the scroll, which is ok for our current
                         // needs, but needs investigation if we will be using this function
                         // in other places.
-                        offset += parseInt( this.GetCurrentElementStyle( element, 'marginBottom' ) || 0, 10 ) ;
+                        var margin = parseInt( this.GetCurrentElementStyle( element, 'marginBottom' ) || 0, 10 ) ;
+                        if ( !isNaN( margin ) )
+                                offset += margin ;
                 }
 
                 // Appends the offsets for the entire element hierarchy.
-                offset += element.offsetTop ;
-                while ( ( element = element.offsetParent ) )
-                        offset += element.offsetTop || 0 ;
+                var elementPosition = FCKTools.GetDocumentPosition( window, element ) ;
+                offset += elementPosition.y ;
 
                 // Scroll the window to the desired position, if not already visible.
                 var currentScroll = FCKTools.GetScrollPosition( window ).Y ;

editor/dialog/fck_replace.html

@@ -185 +185 @@
                 var focusNode = domRange.StartNode ;
                 if ( focusNode.nodeType != 1 )
                         focusNode = focusNode.parentNode ;
-                focusNode.scrollIntoView( false ) ;
+                FCKDomTools.ScrollIntoView( focusNode, false ) ;
                 this._cursors = charRange._cursors ;
         },