Namespace collision in LayoutDialog?

[ncubeait]

Hi Jack,

Great work with this library - I am so impressed!

Anyway, I believe I detected a namespace collision that affects the LayoutDialog when using a center layout region.

Basically, adding such a region overwrites the center() function of the BasicDialog, which in turn breaks things like resizeTo() when fixedcenter:true.

The problem seems to result from merging the layout configuration with the dialog configuration. Perhaps it would be best to keep the two as separate objects.

Will.

[tryanDLS]

Can you post a sample? Are you using the latest code (RC3)?

I'm using a pretty complex 5 panel layout with both contentpanels and gridpanels inside and a resizable Dialog without problems.

[Animal]

No, that sounds feasible. The BasicDialog constructor does a



YAHOO.ext.util.Config.apply(this, config);


Which, if there is a "center" property in the config will create a "center" property in that instance of BasicDialog.

Perhaps the method should be changed to centerDialog.

[tryanDLS]

Yeah, but that could be said of any 'property' that somebody dumps into config. What if I say addButton:'myBtn'. That's going to hammer the addButton method.

Am I missing something here??

[Animal]

Yes and no. The thing is, "center" is a valid config property and an existing method.

[tryanDLS]

Isn't the property name 'fixedcenter'?

[Animal]

A LayoutDialog inherits the setup config of a BorderLayout as well as the config of a BasicDialog. ie regions. One of which is "center". It also has a center() method.

[tryanDLS]

Ahh...gotcha - adding to my list for Jack.