core/Element.js anchorTo() broken chain

[papasi]

Hi,

This is minor issue but fixing it makes it a lot more convenient to chain Element's method call like


Ext.get("error").setWidth(300).anchorTo('dialog', 'b-b').hide().show();


Here's a simple patch:


anchorTo : function(el, alignment, offsets, animate, monitorScroll){
var action = function(){
this.alignTo(el, alignment, offsets, animate);
};
Ext.EventManager.onWindowResize(action, this);
var tm = typeof monitorScroll;
if(tm != 'undefined'){
Ext.EventManager.on(window, 'scroll', action, this,
{buffer: tm == 'number' ? monitorScroll : 50});
}
action.call(this); // align immediately
},



anchorTo : function(el, alignment, offsets, animate, monitorScroll){
var action = function(){
return this.alignTo(el, alignment, offsets, animate);
};
Ext.EventManager.onWindowResize(action, this);
var tm = typeof monitorScroll;
if(tm != 'undefined'){
Ext.EventManager.on(window, 'scroll', action, this,
{buffer: tm == 'number' ? monitorScroll : 50});
}
return action.call(this); // align immediately
},

Thanks

[papasi]

Hi,

I notice a few more broken chain in widgets/Components.js while using the form stuff.

focus() , blur(), disable(), enable() etc.

Should be easy to fix.

Thanks.

[jack.slocum]

Chaining is not required, therefore is not a bug. I would be happy to add these things in tho. ;)