[SOLVED] Node Lister - minor bug

[nat.harrold]

Hi Again,

 

Minor bug (but still annoying),

 

"Move Node up in the List" button misbehaves.

 

 

When having a node selected that you want to move up the tree, clicking the "move node up list" button multiple times can result in your oriiginal selection changing to a random object and moving that instead.

 

This happens if you click the "move node up list" button faster than the node can regenerate its list position.

 

Thanks,

Nat.

[necris]

Issue was reproduced and passed to developers.

 

Thanks!

[frustum]

Please apply this patch to data/core/editor/editor_nodes.h file at 1430 line:

         // select node again properly
-        reselect_node();
+        nodes_tb.setCallbackEnabled(GUI_CHANGED,0);
+        foreachkey(int id; current_nodes) {
+            if(current_nodes[id] == node) {
+                nodes_tb.setItemSelected(id,1);
+                nodes_tb.showItem(id);
+            } else {
+                nodes_tb.setItemSelected(id,0);
+            }
+        }
+        nodes_tb.setCallbackEnabled(GUI_CHANGED,1);