Hide Scrollbar for my autogrow functionality

as they are config options then it should be possible to state them in the Replace call and it will override these options. That was my goal anyway but you are right.. I messed it up a little. So I think I have fixed it using the editor.config rather than the CKEDITOR.config. Seems to work for me.
So the default values are in the plugin but they will be overwritten if you set them in the replace

Thank you so much for the input

(function()
{
   CKEDITOR.plugins.autogrow =
   {
      getHeightDelta : function( editor )
      {
         var contents = CKEDITOR.document.getById( 'cke_contents_' + editor.name );
         var outer = contents.getAscendant( 'table' ).getParent().getParent().getParent();
     
         // Get the height delta between the outer table and the content area.
         var delta =  ( outer.$.offsetHeight || 0 ) - ( contents.$.clientHeight || 0 );
         return delta;
      },
     
      getEffectiveHeight : function( height, maxheight, minheight )
      {
         if ( height < minheight )
            height = minheight;
         else
         {
            if ( maxheight && maxheight > 0 && height > maxheight )
               height = maxheight;
         }
     
         return height;
      }
   };
   function checkSize ( evt )
   {
      var editor = evt.editor;
      var delta = plugin.getHeightDelta(editor) ;
      var maxheight = editor.config.autogrow_maxheight;
      var minheight = editor.config.autogrow_minheight;
      var $body = editor.document.$.body;
      if(delta != 0){
         var newHeight = delta + $body.offsetHeight;
         //add 10 as if you remove Plugin resize then it is a little too short
         newHeight = plugin.getEffectiveHeight( newHeight , maxheight, minheight ) + 10 ;
         var contents = CKEDITOR.document.getById( 'cke_contents_' + editor.name );
         newHeight = Math.max( newHeight, 0 ) + 'px';
         if(contents.getStyle('height') != newHeight)
         {
            contents.setStyle( 'height', newHeight );
         }
      }
   };
      
   var plugin = CKEDITOR.plugins.autogrow;
   
   CKEDITOR.plugins.add( 'autogrow',
   {
      init : function( editor )
      {
               editor.on( 'insertHtml', checkSize );
               editor.on( 'insertElement', checkSize );
               editor.on( 'selectionChange', checkSize );
               editor.on( 'instanceReady', checkSize );
      }
   } );
})();

CKEDITOR.config.autogrow_maxheight=0;
CKEDITOR.config.autogrow_minheight=50;

yeah... it is a little bit of a hack as I found that when I removed the resize plugin (because it wasn't necessary as my area was auto growing) it messed up all my sizes and everything was a little too short and I was getting vertical scrollbars.

However, if you are not removing the resize plugin then you could remove this 10px.

What I really should have done is either
a) done a check to see if the resize plugin was active or not
b) find the magic id that would give me the right height in the first place.

I will have a look at this this evening and see if I can do one of the above...

Sorry about the delay in responding. You try and fix one thing and you notice 3 others... argh..

anyhow.. I have tweaked autogrow so you can provide your own padding from config if you need it
I have also changed it so it still works if the delta value =0 (which occurs when you hack the style sheet to remove all the padding you don't really want and removePlugins: resize,elementspath)

so it still isn't perfect as I would prefer not to have to hack some padding in.. but I am just not sure how else to do it. But at least you have the option to change the value/set to 0 if you need to

(function()
{
   CKEDITOR.plugins.autogrow =
   {
      getHeightDelta : function( editor )
      {
         var contents = CKEDITOR.document.getById( 'cke_contents_' + editor.name );
         var outer = contents.getAscendant( 'table' ).getParent().getParent().getParent();
     
         // Get the height delta between the outer table and the content area.
         var delta =  ( outer.$.offsetHeight || 0 ) - ( contents.$.clientHeight || 0 );
         return delta;
      },
     
      getEffectiveHeight : function( height, maxheight, minheight, padding )
      {
         var totalheight = parseInt(height,10) + parseInt(padding,10);
         if ( totalheight < minheight )
            totalheight = minheight;
         else
         {
            if ( maxheight && maxheight > 0 && totalheight > maxheight )
               totalheight = maxheight;
         }
     
         return totalheight;
      }
   };
   function checkSize ( evt )
   {
      var editor = evt.editor;
      var delta = plugin.getHeightDelta(editor) ;
      var maxheight = editor.config.autogrow_maxheight  ;
      var minheight = editor.config.autogrow_minheight ;
      var padding = editor.config.autogrow_padding ;
      var $body = editor.document.$.body;
      var newHeight = delta + $body.offsetHeight;
      //add a little padding as sometimes it just doesn't go big enough
      //especially if you remove Plugin resize
      newHeight = plugin.getEffectiveHeight( newHeight , maxheight, minheight, padding );
      var contents = CKEDITOR.document.getById( 'cke_contents_' + editor.name );
      newHeight = Math.max( newHeight, 0 ) + 'px';
      if(contents.getStyle('height') != newHeight)
      {
         contents.setStyle( 'height', newHeight );
      }
   };
      
   var plugin = CKEDITOR.plugins.autogrow;
   
   CKEDITOR.plugins.add( 'autogrow',
   {
      init : function( editor )
      {
               editor.on( 'insertHtml', checkSize );
               editor.on( 'insertElement', checkSize );
               editor.on( 'selectionChange', checkSize );
               editor.on( 'instanceReady', checkSize );
      }
   } );
})();

CKEDITOR.config.autogrow_maxheight=0;
CKEDITOR.config.autogrow_minheight=50;
CKEDITOR.config.autogrow_padding = 30;

In IE the height numbers are out, so it puts a vertical scrollbar in as it gets confused... It might be to do with it allowing space for the horizontal scrollbar or some such other issue. But basically I got bored of trying to find it.
It all worked fine when I still had the plugins resize and elementspath, but removing them just upset it, so the vertical padding was added (basically it just extends the editable area to a tad longer than it needs for the content in it). but if you don't need it. Don't include it.

Also sometimes a little extra space at the bottom looks nice and it has the added advantage that the vertical scrollbar doesn't flicker in and out as much...

Would you believe it but my editor isn't actually finished yet. Still have some magic to weave. The main bits are there but it kinda looks like crap at the moment so need to make it pretty. Basically I had a WYSIWYG editor based on fckeditor 2.0 and I foolishly decided to upgrade it, which actually turned into a complete rewrite and if I ever finish it.. It will be better than the original (mainly because scope creep crept in)