Sebastian Werner <[EMAIL PROTECTED]> writes:
> * Keyboard-navigation seems to be buggy
I could use some help with what qooxdoo is doing to cause this. The attached
patch shows the reason for the problem: moving around the tree, e.g. to the
root, and then back down cause the selection manager's
renderItemSelectionState() function to be called with vItem set to something
that's not in the tree! Instead, vItem is a GroupBox, which does exist in the
example, but is outside of the tree and has no setSelected() method.
To see what I'm seeing, apply this patch, click on the root, and press the
down arrow... (Making a similar change to TreeSelectionManager.js shows that
in the Tree_1.html example, vItem is always something in the tree.)
===
/home/derrell/backlot/future-tests/qooxdoo/branches/namespaces/qooxdoo/source/script/qx/manager/selection/TreeFullControlSelectionManager.js
==================================================================
---
/home/derrell/backlot/future-tests/qooxdoo/branches/namespaces/qooxdoo/source/script/qx/manager/selection/TreeFullControlSelectionManager.js
(revision 15692)
+++
/home/derrell/backlot/future-tests/qooxdoo/branches/namespaces/qooxdoo/source/script/qx/manager/selection/TreeFullControlSelectionManager.js
(local)
@@ -225,5 +225,6 @@
*/
qx.Proto.renderItemSelectionState = function(vItem, vIsSelected) {
+ this.debug("vItem=" + vItem);
vItem.setSelected(vIsSelected);
};
_______________________________________________
Qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
