Derrell Lipman schrieb: > On Mon, Jul 28, 2008 at 10:40 AM, Fabian Jakobs > <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> wrote: > > 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. > > > Ok, I'll have to look at how that qx.ui.core.LayoutItem stuff works. > > qx.ui.util.column.Data extended Widget in order to be able to do the > flex calculations. Widget already had all of the fields that those > calculations used. I'm not sure, therefore, that > qx.ui.util.column.FlexWidth is obsolete; rather, it may just need to > be updated to use qx.ui.core.LayoutItem if flex calculations need be > done in other widgets which require column width determination (such > as Progressive's table). > > Do you think you could take a quick look at the function > _resizeColumns() in qx/ui/progressive/renderer/table/Row.js and let me > know how that might be done in 0.8? > From first sight it seems straightforward to use the same strategy for your progressive table as well. You can take a look at "qx.ui.table.columnmodel.resizebehavior.Default" of how I have done it in the resize table model. The changes are mainly in "_computeColumnsFlexWidth".
Best Fabian -- Fabian Jakobs JavaScript Framework Developer 1&1 Internet AG Brauerstraße 48 76135 Karlsruhe Amtsgericht Montabaur HRB 6484 Vorstand: Henning Ahlert, Ralph Dommermuth, Matthias Ehrlich, Thomas Gottschlich, Matthias Greve, Robert Hoffmann, Markus Huhn, Oliver Mauss, Achim Weiss Aufsichtsratsvorsitzender: Michael Scheeren ------------------------------------------------------------------------- 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
