Ticket #2127: 2127.patch

_whatsnew.html

@@ -51 +51 @@
                         which stopped working in FCKeditor 2.6.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2039">#2039</a>] Fixed the locking up issue
                         in the Find/Replace dialog.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/2127">#2127</a>] Window scrollbars are now
+                        disabled when floating dialogs are being displayed.</li>
         </ul>
         <h3>
                 Version 2.6</h3>

editor/_source/internals/fckdialog.js

@@ -26 +26 @@
         var topDialog ;
         var baseZIndex ;
         var cover ;
+        var previousOverflow ;
+        var scrollElement ;
 
         // The document that holds the dialog.
         var topWindow = window.parent ;
@@ -190 +192 @@
                                 } ) ;
                         FCKDomTools.SetOpacity( cover, FCKConfig.BackgroundBlockerOpacity ) ;
 
+                        // Hide the scrollbar of the top window.
+                        scrollElement = topDocument.compatMode == 'CSS1Compat' ? topDocument.documentElement : topDocument.body ;
+                        previousOverflow = scrollElement.style.overflow ;
+                        var previousScroll = FCKTools.GetScrollPosition( topWindow ) ;
+                        scrollElement.style.overflow = 'hidden' ;
+                        scrollElement.scrollLeft = previousScroll.X ;
+                        scrollElement.scrollTop = previousScroll.Y ;
+
                         // For IE6-, we need to fill the cover with a transparent IFRAME,
                         // to properly block <select> fields.
                         if ( FCKBrowserInfo.IsIE && !FCKBrowserInfo.IsIE7 )
@@ -224 +234 @@
                 {
                         FCKDomTools.RemoveNode( cover ) ;
                         FCKFocusManager.Unlock() ;
+
+                        // Restore the scrollbar of the top window to its previous state.
+                        scrollElement.style.overflow = previousOverflow ;
                 },
 
                 GetCover : function()