Fix GitHub Issue #87: Group move doesn't work with the move handle

juzzlin · Jul 4, 2020 · 65be2ee · 65be2ee
1 parent 3f6609f
commit 65be2ee
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -5,6 +5,8 @@ New features:
 
 Bug fixes:
 
+* Fix GitHub Issue #87: Group move doesn't work with the move handle
+
 Other:
 
 1.18.0

diff --git a/src/editor_view.cpp b/src/editor_view.cpp
@@ -123,8 +123,6 @@ void EditorView::handleMousePressEventOnNodeHandle(QMouseEvent & event, NodeHand
         return;
     }
 
-    m_mediator.clearSelectionGroup();
-
     if (event.button() == Qt::LeftButton) {
         handleLeftButtonClickOnNodeHandle(nodeHandle);
     }
@@ -150,16 +148,19 @@ void EditorView::handleLeftButtonClickOnNodeHandle(NodeHandle & nodeHandle)
 {
     switch (nodeHandle.role()) {
     case NodeHandle::Role::Add:
+        m_mediator.clearSelectionGroup();
         initiateNewNodeDrag(nodeHandle);
         break;
     case NodeHandle::Role::Drag:
         initiateNodeDrag(nodeHandle.parentNode());
         break;
     case NodeHandle::Role::Color:
+        m_mediator.clearSelectionGroup();
         m_mediator.setSelectedNode(&nodeHandle.parentNode());
         openNodeColorDialog();
         break;
     case NodeHandle::Role::TextColor:
+        m_mediator.clearSelectionGroup();
         m_mediator.setSelectedNode(&nodeHandle.parentNode());
         openNodeTextColorDialog();
         break;