X-Git-Url: http://git.roojs.org/?a=blobdiff_plain;f=Roo%2FBasicDialog.js;h=a4d6ab1e308141b23cb73ee79180bbe9ba25bac7;hb=0b69a11b1055a6d3a3fd0719bdd58b5a235de0e4;hp=9c26ab7349e91ac0f52341793f418f9de5a2f104;hpb=3676af4483523e67a2490f62d9754455f6582ec8;p=roojs1

diff --git a/Roo/BasicDialog.js b/Roo/BasicDialog.js
index 9c26ab7349..a4d6ab1e30 100644
--- a/Roo/BasicDialog.js
+++ b/Roo/BasicDialog.js
@@ -556,15 +556,20 @@ Roo.extend(Roo.BasicDialog, Roo.util.Observable, {
     },
 
     // private
-    syncBodyHeight : function(){
-        var bd = this.body, cb = this.centerBg, bw = this.bwrap;
+    syncBodyHeight : function()
+    {
+        var bd = this.body, // the text
+            cb = this.centerBg, // wrapper around bottom.. but does not seem to be used..
+            bw = this.bwrap;
         var height = this.size.height - this.getHeaderFooterHeight(false);
         bd.setHeight(height-bd.getMargins("tb"));
         var hh = this.header.getHeight();
         var h = this.size.height-hh;
         cb.setHeight(h);
+        
         bw.setLeftTop(cb.getPadding("l"), hh+cb.getPadding("t"));
         bw.setHeight(h-cb.getPadding("tb"));
+        
         bw.setWidth(this.el.getWidth(true)-cb.getPadding("lr"));
         bd.setWidth(bw.getWidth(true));
         if(this.tabs){
@@ -1177,7 +1182,8 @@ Roo.LayoutDialog = function(el, cfg){
         config = Roo.apply({}, el);
         // not sure why we use documentElement here.. - it should always be body.
         // IE7 borks horribly if we use documentElement.
-        el = Roo.get( Roo.isIE ? (document.body || document.documentElement) : (document.documentElement || document.body) ).createChild();
+        // webkit also does not like documentElement - it creates a body element...
+        el = Roo.get( document.body || document.documentElement ).createChild();
         //config.autoCreate = true;
     }