On Tue, Feb 2, 2010 at 01:41, Tom Schindl <tom.schi...@bestsolution.at>wrote:

> Hi,
>
> Thanks. That solved the problem though I still had to fight a bit
> because the virtual-API looks like has a flaw because the order of API
> calls is important.
>
> Background to this is that I assign my own SimpleTreeDataCellRenderer
> because I don't want to show icons at all.
>
> So my first setup looks like this:
>
> --------8<--------
> tree.setAlwaysShowOpenCloseSymbol(false);
> tree.setExcludeFirstLevelTreeLines(true);
> tree.getTableColumnModel().setDataCellRenderer(0, renderer);
> --------8<--------
>
> But this is not using this working when assigning a new renderer because
> the attributes in reality stored in the renderer itself and when setting
> custom one they are reset to the default.
>
> I think this has to be documented in the API at least and IMHO the
> method should be deprecated in favor of the ones in
> SimpleTreeDataCellRenderer.
>

Thanks for this! I looked at documenting it, but it would be relevant only
to very few people and would confuse the issue. This actually affects many
functions, as they are shorthand for retrieving the cell renderer and
operating on it. I disagree about deprecating the methods in TreeVirtual,
though. No one, during normal usage (your usage is not normal) should have
to know or care about the internal workings of the widget. For those of us
who do really fancy things like altering the tree cell renderer, we need to
look at the source code and get familiar with the code. Making changes to
the cell renderer requires looking at the code, so the people who would need
to do this are already familiar with doing so.

Thanks again!

Derrell
------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to