PDA

View Full Version : [FIXED]Measurement in VBoxLayout typo



KampfCaspar
21 Apr 2009, 10:00 PM
BoxLayout.js#151 in rc1:

maxWidth = Math.max(maxWidth, c.getWidth() + cm.left + cm.top);


I suppose, this should be cm.right for the WIDTH?

mjlecomte
22 Apr 2009, 5:29 AM
Not a lot of info here to support why you think this is a bug.

Condor
22 Apr 2009, 6:38 AM
Are you really expecting a full bugreport for such an obvious bug?

Ext.layout.HBoxLayout.onLayout contains:

maxHeight = Math.max(maxHeight, c.getHeight() + cm.top + cm.bottom);

while Ext.layout.VBoxLayout.onLayout contains:

maxWidth = Math.max(maxWidth, c.getWidth() + cm.left + cm.top);

ps. The title is wrong however. It's a problem in VBoxLayout and not in HBoxLayout.

mjlecomte
22 Apr 2009, 6:59 AM
Your post makes it obvious, the OP's didn't.

evant
22 Apr 2009, 9:49 AM
Fixed in SVN, thanks.