Merge pull request #249 from cneben/f/#244-navigable-polish

F/#244 navigable polish

Author: cneben

src/qanDraggableCtrl.cpp

@@ -226,6 +226,10 @@ void    DraggableCtrl::beginDragMove(const QPointF& sceneDragPos, bool dragSelec
     _targetItem->setDragged(true);
 
     _initialSceneDragPos = sceneDragPos;
+    if (!_target->isGroup()) {
+        _initialTargetZ = _targetItem->z();     // 20240821: Force maximum graph z when dragging a node (not a group), restored
+        _targetItem->setZ(graph->getMaxZ() + 10.);    // in endDragMove
+    }
     const auto rootItem = getGraph()->getContainerItem();
     if (rootItem != nullptr)   // Project in scene rect (for example if a node is part of a group)
         _initialTargetScenePos = rootItem->mapFromItem(_targetItem, QPointF{0,0});
@@ -419,6 +423,11 @@ void    DraggableCtrl::endDragMove(bool dragSelection, bool notify)
         !_targetItem)
         return;
 
+    if (!_target->isGroup()) {
+        _targetItem->setZ(_initialTargetZ);
+        _initialTargetZ = 0.;
+    }
+
     if (_target->getIsProtected() ||    // Prevent dragging of protected or locked objects
         _target->getLocked())
         return;

src/qanDraggableCtrl.h

@@ -113,6 +113,8 @@ class DraggableCtrl : public qan::AbstractDraggableCtrl
     QPointF                 _initialSceneDragPos{0., 0.};
     //! Internal (target) initial dragging position.
     QPointF                 _initialTargetScenePos{0., 0.};
+    //! Internal (target) initial dragging z.
+    double                  _initialTargetZ = 0.;
 
     //! Last group hovered during a node drag (cached to generate a dragLeave signal on qan::Group).
     QPointer<qan::Group>    _lastProposedGroup{nullptr};