It works!
Applied your patch both to legacy 0.7 branch and trunk

Derrell Lipman wrote:
> 
> Ok, ease-of-use and lack of confusion trumps a bit of extra memory. 
> David,
> please try the attached (untested) patch and let me know how it works. 
> Note
> that this is against legacy_0_7_x to which I applied another small change
> earlier, so you should "svn up" before applying the patch.  Once you
> confirm
> that it works properly, I'll check it in.
> 
> Cheers,
> 
> Derrell
> 
> Index:
> frontend/framework/source/class/qx/ui/treevirtual/SimpleTreeDataModel.js
> ===================================================================
> ---
> frontend/framework/source/class/qx/ui/treevirtual/SimpleTreeDataModel.js
> (revision 10333)
> +++
> frontend/framework/source/class/qx/ui/treevirtual/SimpleTreeDataModel.js
> (working copy)
> @@ -417,6 +417,7 @@
>        var node =
>        {
>          type           : type,
> +        nodeId         : nodeId,
>          parentNodeId   : parentNodeId,
>          label          : label,
>          bSelected      : false,
> @@ -637,10 +638,6 @@
>                continue;
>              }
>  
> -            // Listeners will need to know a node's id when they receive
> an
> -            // event.
> -            child.nodeId = childNodeId;
> -
>              // (Re-)assign this node's level
>              child.level = level;
> 

-- 
View this message in context: 
http://www.nabble.com/nodeId-missing-in-a-TreeVirtual-tf4528232.html#a12938014
Sent from the qooxdoo-devel mailing list archive at Nabble.com.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to