Ticket #8050: 8050_7.patch

_source/plugins/autogrow/plugin.js

@@ -7 +7 @@
  * @file AutoGrow plugin
  */
 (function(){
+
+        // Actual content height, figured out by appending check the last element's document position.
+        function contentHeight( doc )
+        {
+                // Create a temporary marker element.
+                var marker = CKEDITOR.dom.element.createFromHtml( '<span style="margin:0;padding:0;border:0;clear:both;width:1px;height:1px;display:block;">' + ( CKEDITOR.env.webkit ? '&nbsp;' : '' ) + '</span>', doc );
+                var body = doc.getBody();
+                body.append( marker );
+
+                var height = marker.getDocumentPosition( doc ).y + marker.$.offsetHeight;
+                marker.remove();
+                return height;
+        }
+
         var resizeEditor = function( editor )
         {
                 if ( !editor.window )
                         return;
+
                 var doc = editor.document,
+                        body = doc.getBody(),
+                        htmlElement = doc.getDocumentElement(),
                         currentHeight = editor.window.getViewPaneSize().height,
-                        newHeight;
+                        newHeight = contentHeight( doc ),
+                        // Quirks mode overflows body, standards overflows document element
+                        scrollable = doc.$.compatMode == 'BackCompat' ? body : htmlElement;
 
-                // We can not use documentElement to calculate the height for IE (#6061).
-                // It is not good for IE Quirks, yet using offsetHeight would also not work as expected (#6408).
-                // We do the same for FF because of the html height workaround (#6341).
-                if ( CKEDITOR.env.ie || CKEDITOR.env.gecko )
-                        newHeight = doc.getBody().$.scrollHeight + ( CKEDITOR.env.ie && CKEDITOR.env.quirks ? 0 : 24 );
-                else
-                        newHeight = doc.getDocumentElement().$.offsetHeight;
+                // Minor adjustment for each browser, or specified by user.
+                newHeight += ( editor.config.autoGrow_bottomSpace !== undefined || ( CKEDITOR.env.ie9Compat ? 0 : CKEDITOR.env.ie || CKEDITOR.env.opera ? -20 :  20 ) );
 
-                var min = editor.config.autoGrow_minHeight,
-                        max = editor.config.autoGrow_maxHeight;
-                ( min == undefined ) && ( editor.config.autoGrow_minHeight = min = 200 );
-                if ( min )
-                        newHeight = Math.max( newHeight, min );
-                if ( max )
-                        newHeight = Math.min( newHeight, max );
+                var min = editor.config.autoGrow_minHeight || 200,
+                        max = editor.config.autoGrow_maxHeight || Infinity;
+
+                newHeight = Math.max( newHeight, min );
+                newHeight = Math.min( newHeight, max );
 
                 if ( newHeight != currentHeight )
                 {
                         newHeight = editor.fire( 'autoGrow', { currentHeight : currentHeight, newHeight : newHeight } ).newHeight;
                         editor.resize( editor.container.getStyle( 'width' ), newHeight, true );
+
+                        if ( newHeight < max )
+                                scrollable.setStyle( 'overflow-y', 'hidden' );
+                        else
+                                scrollable.removeStyle( 'overflow-y' );
                 }
         };
         CKEDITOR.plugins.add( 'autogrow',
@@ -86 +103 @@
  * @param {Number} data.newHeight The new height of the editor (after the resizing). It can be changed
  *                              to determine another height to be used instead.
  */
+
+
+/**
+ *  Extra height in pixel to leave between the bottom boundary of content with document size when auto resizing.
+ * @name CKEDITOR.config.autoGrow_bottomSpace
+ * @type Number
+ * @default 20px
+ * @since 3.6.2
+ */