[FIXED][3.0rc1] show() with an Element hides the buttons

[stever]

Try this on both IE7 (or IE8 in compatibility mode) and Firefox:


new Ext.Window({title:'test', height:200,width:300, items:{height:40},buttons:[{text:'t1'},{text:'t2'}]}).show(Ext.getBody())

In IE7, the buttons do not get displayed.

[stever]

Seems that the buttons are also hidden in Safari (and Chrome?).

[Animal]

This must be fixed in SVN because I can't reproduce it in any examples pages in my development directory.

[evant]

I think I might have committed a fix for this.

[aconran]

Looks resolved. Steve R, could you please confirm?

[stever]

Not fixed. I just tried with the SVN of this minute, opened up IE8, hit compatibility mode button, and added the example into my local build and got this again:





new Ext.Window({title:'test', height:200,width:300, items:{height:40},buttons:[{text:'t1'},{text:'t2'}]}).show(Ext.getBody())

[evant]

Confirmed, it only happens with animations. Try the following:



Ext.override(Ext.Window, {
afterShow : function(isAnim){
this.proxy.hide();
this.el.setStyle('display', 'block');
this.el.show();
if(this.maximized){
this.fitContainer();
}
if(Ext.isMac && Ext.isGecko){ // work around stupid FF 2.0/Mac scroll bar bug
this.cascade(this.setAutoScroll);
}

if(this.monitorResize || this.modal || this.constrain || this.constrainHeader){
Ext.EventManager.onWindowResize(this.onWindowResize, this);
}
this.doConstrain();
if(this.layout){
this.doLayout();
}
if(this.keyMap){
this.keyMap.enable();
}
this.toFront();
this.updateHandles();
if(isAnim && (Ext.isIE || Ext.isWebKit)){
var sz = this.getSize();
this.onResize(sz.width, sz.height);
}
this.fireEvent("show", this);
},

// private
animShow : function(){
this.proxy.show();
this.proxy.setBox(this.animateTarget.getBox());
this.proxy.setOpacity(0);
var b = this.getBox(false);
b.callback = this.afterShow.createDelegate(this, [true], false);
b.scope = this;
b.duration = .25;
b.easing = 'easeNone';
b.opacity = .5;
b.block = true;
this.el.setStyle('display', 'none');
this.proxy.shift(b);
}
});

[stever]

The fix that is in SVN works great. Thanks!