Hi Derrell,
> Hi Fabian,
>
> Be careful about where you put this stuff from ui.util.column.  I had 
> split it out from Table because it is useful for other classes.  In 
> particular, it's currently also used by Progressive's table.
>
> Cheers,
>
> Derrell

I have modified the resize column renderer to use the new layout manager 
to calculate the flex and percent values. I was not entirely sure to 
what extend you depended on the fact that qx.ui.util.column.Data extends 
Widget. Since the widget interface has changed considerably It would 
have been hard to just fix it. If I understand your intentions correct, 
you did this mainly to have all those layout properties, like height, 
minHeight, maxHeight, ... . In this sense I replaced this class with a 
new class, which derived from qx.ui.core.LayoutItem. It has all the 
layout relevant properties as well and can be put into a layout manager. 
The nice thing about the new layout system is that it can be used 
independently of the rest of the widget system. I now use a standard 
qooxdoo box layout to do the percent and flex computations. For this 
reason qx.ui.util.column.FlexWidth has become obsolete and since 
qx.ui.data.column.Widths is now used in the table I removed the "data" 
namespace and put the "qx.ui.util.column.Data" class back into the table 
namespace.

Right now the "Data" code is only used by the table and the other code 
isn't used at all in 0.8. Once you port the progressive table and feel 
its useful to have this class outside of the table we can still move it 
back.

I hope the changes are in your sense.

Best Fabian


P.S.

The relevant commit is rev. 14887





-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to