On 08/11/10 17:17, Derrell Lipman wrote:
> On Wed, Aug 11, 2010 at 06:22, Mengu <[email protected]
> <mailto:[email protected]>> wrote:
>
>
>     Hello Derrell,
>
>     Thank you for the patch. We have applied the patch. The problem is
>     "isMouseAction" that comes from e.getData() is undefined. 
>
>
> Ok, I had a bit more time today so I actually tested it this time. :-)
> Try attached patch x.patch instead, or apply y.patch on top of what I
> gave you yesterday.
>
> Derrell
>

derrell,

fyi, in order to achieve the same goal with orderChanged event, i made
similar additions to the code in:

1) qx.ui.table.columnmodel.Basic.moveColumn, adding a isMouseAction argument
2) same function, adding a
         isMouseAction: isMouseAction || false

line to the data object.

3) qx.ui.table.pane.Scroller._stopMoveHeader, adding true as third
argument to its moveColumn call.

what do you think?

also, there's a note in qx.ui.table.columnmodel.Basic.setColumnsOrder

        // Inform the listeners. Do not add data as all known listeners
in qooxdoo
        // only take this event to mean "total repaint necesscary".
Fabian will look
        // after deprecating the data part of the orderChanged - event
        this.fireDataEvent("orderChanged");

there doesn't seem to be a bug open for it.

best,
burak



------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to