Roo/Template.js

[roojs1] / Roo / BasicDialog.js

diff --git a/Roo/BasicDialog.js b/Roo/BasicDialog.js
index a5491be..e4df15e 100644 (file)
--- a/Roo/BasicDialog.js
+++ b/Roo/BasicDialog.js
@@ -604,7 +604,7 @@ Roo.extend(Roo.BasicDialog, Roo.util.Observable, {
 
     // private
     animShow : function(){
-        var b = Roo.get(this.animateTarget, true).getBox();
+        var b = Roo.get(this.animateTarget).getBox();
         this.proxy.setSize(b.width, b.height);
         this.proxy.setLocation(b.x, b.y);
         this.proxy.show();
@@ -630,7 +630,7 @@ Roo.extend(Roo.BasicDialog, Roo.util.Observable, {
         this.animateTarget = animateTarget || this.animateTarget;
         if(!this.el.isVisible()){
             this.beforeShow();
-            if(this.animateTarget){
+            if(this.animateTarget && Roo.get(this.animateTarget)){
                 this.animShow();
             }else{
                 this.showEl();
@@ -934,7 +934,9 @@ Roo.extend(Roo.BasicDialog, Roo.util.Observable, {
         if(this.shim) {
           this.shim.hide();
         }
-        if(this.animateTarget){
+        // sometimes animateTarget seems to get set.. causing problems...
+        // this just double checks..
+        if(this.animateTarget && Roo.get(this.animateTarget)) {
            this.animHide(callback);
         }else{
             this.el.hide();
@@ -1173,7 +1175,10 @@ Roo.LayoutDialog = function(el, cfg){
     var config=  cfg;
     if (typeof(cfg) == 'undefined') {
         config = Roo.apply({}, el);
-        el = Roo.get( document.body || document.documentElement).createChild();
+        // not sure why we use documentElement here.. - it should always be body.
+        // IE7 borks horribly if we use documentElement.
+        // webkit also does not like documentElement - it creates a body element...
+        el = Roo.get( document.body || document.documentElement ).createChild();
         //config.autoCreate = true;
     }