From 158bb47af31c28071aebdf5c182bf68130f1fe39 Mon Sep 17 00:00:00 2001
From: Filip Sykala <filip.sykala@prusa3d.cz>
Date: Thu, 19 Oct 2023 14:41:51 +0800
Subject: [PATCH] Suggestion to detect volume modification instead of check
 some gizmos during dragging

(cherry picked from commit prusa3d/PrusaSlicer@96610ecea98a5be335ed0664d2da9280e9658edb)
---
 src/slic3r/GUI/GLCanvas3D.cpp | 42 +++++++++++++++--------------------
 1 file changed, 18 insertions(+), 24 deletions(-)

diff --git a/src/slic3r/GUI/GLCanvas3D.cpp b/src/slic3r/GUI/GLCanvas3D.cpp
index 206747ce80..d287113a28 100644
--- a/src/slic3r/GUI/GLCanvas3D.cpp
+++ b/src/slic3r/GUI/GLCanvas3D.cpp
@@ -3900,30 +3900,24 @@ void GLCanvas3D::on_mouse(wxMouseEvent& evt)
         m_mouse.set_start_position_3D_as_invalid();
         m_mouse.position = pos.cast<double>();
 
-        if (evt.Dragging() && current_printer_technology() == ptFFF && (fff_print()->config().print_sequence == PrintSequence::ByObject)) {
-            switch (m_gizmos.get_current_type())
-            {
-            case GLGizmosManager::EType::Move:
-            case GLGizmosManager::EType::Scale:
-            case GLGizmosManager::EType::Rotate:
-            {
-                update_sequential_clearance();
-                break;
-            }
-            default: { break; }
-            }
-        }
-        else if (evt.Dragging()) {
-            switch (m_gizmos.get_current_type())
-            {
-            case GLGizmosManager::EType::Move:
-            case GLGizmosManager::EType::Scale:
-            case GLGizmosManager::EType::Rotate:
-            {
-                show_sinking_contours();
-                break;
-            }
-            default: { break; }
+        // It should be detection of volume change
+        // Not only detection of some modifiers !!!
+        if (evt.Dragging()) {
+            if (current_printer_technology() == ptFFF &&
+                (fff_print()->config().print_sequence == PrintSequence::ByObject)) {
+                switch (m_gizmos.get_current_type()) {
+                case GLGizmosManager::EType::Move:
+                case GLGizmosManager::EType::Scale:
+                case GLGizmosManager::EType::Rotate:
+                    update_sequential_clearance();
+                }
+            } else {
+                switch (m_gizmos.get_current_type()) {
+                case GLGizmosManager::EType::Move:
+                case GLGizmosManager::EType::Scale:
+                case GLGizmosManager::EType::Rotate: 
+                    show_sinking_contours();
+                }
             }
         }
         else if (evt.LeftUp() &&