🚀 The bring_forward method now works in constant time (or log if yo…

…u count the depth of the Qt graphics tree)

opencodeblocks/graphics/view.py

@@ -36,6 +36,7 @@ def __init__(self, scene:OCBScene, parent=None,
 
         self.edge_drag = None
         self.lastMousePos = QPointF(0, 0)
+        self.currentSelectedBlock = None
 
         self.init_ui()
         self.setScene(scene)
@@ -144,14 +145,17 @@ def bring_forward(self, event: QMouseEvent, action="press"):
         scene = self.scene()
         item_at_click = self.itemAt(event.pos())
 
-        while item_at_click.parentItem() != None:
+        while item_at_click.parentItem() is not None:
+            if isinstance(item_at_click,OCBBlock):
+                break
             item_at_click = item_at_click.parentItem()
 
         if isinstance(item_at_click, OCBBlock):
-            for item in scene.items():
-                if isinstance(item,OCBBlock):
-                    item.setZValue(0)
-                item_at_click.setZValue(1)
+            if self.currentSelectedBlock is not None:
+                self.currentSelectedBlock.setZValue(0)
+            item_at_click.setZValue(1)
+            self.currentSelectedBlock = item_at_click
+
         return event # This is never considered as a handling of the event.