On Tue, Jun 15, 2010 at 11:13, panyasan <[email protected]> wrote:
> Derrell Lipman wrote:
> >
> > Yuck. The patches required of TreeVirtual and Table are *fugly*! Some of
> > the patches appear unnecessary, like patching Scroller, where there are
> > already nice provided means to do that sort of thing. In this case, you
> > can
> > provide a function in the 'custom' parameter that instantiates your own
> > subclass of scroller instead of qx.ui.table.pane.Scrolller, to provide
> > those
> > additional methods. Are you aware, though, that there can be multiple
> > scrollers in a single table? Each metacolumn gets its own scroller.
> >
>
> Unfortunately, I don't understand the internals of Table and TreeVirtual
> very well - I have always relied on code snippets that I didn't fully
> grasp,
> which didn't matter since it worked after all. I don't know if saaj is
> still
> on the list, who introduced these patches - but since the patches are very
> small, couldn't you just integrate at least the getter for the private
> properties - this would save us from having to provide custom models to do
> just that.
>
I just looked more carefully at this. It seems reasonable to provide a
getter for the pane scroller. I'll do that. The other one isn't private,
it's protected, and not optimized out by a build. You can call the protected
member from the mixin, so that should be fixed in your code (to call
_getRowForPagePos instead of getRowForPagePos.
Derrell
------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
lucky parental unit. See the prize list and enter to win:
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
