On Mon, Jul 6, 2009 at 6:01 AM, Ian Horst <[email protected]> wrote:
> Derrell,
>
> I see your point. But let's take away from developers to define
> descending function. Qooxdoo framework can figure it out itself. Less
> code is better. :)
>
> I don't have time to look at the backend of qooxdoo right now, but
> instead of setSortMethods we could use
>
> ...setSortMethod: function (sortAscMethod)
> {
> var sortDescMethod = function (row1, row2) {return
> sortAscMethod(row2, row1);};
> var sortMethods = {ascending: sortAscMethod, descending: sortDescMethod};
> ...
> }
>
Ian, that sounds like a great idea. I need to think about it some more to
ensure there aren't cases where simply swapping the parameters wouldn't do
the job, and it is an API change that should be well-considered. Would you
please create an enhancement bug for this, including the issue you're trying
to solve and your proposed solution. That way, this won't get forgotten.
It's likely not to get into the upcoming release, but can still make it into
0.9 if we agree it's appropriate.
Derrell
------------------------------------------------------------------------------
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel