From 7faa61c5dda2b69431926a4658592490d3db68d7 Mon Sep 17 00:00:00 2001
From: "zhou.xu" <zhou.xu@bambulab.com>
Date: Wed, 6 Dec 2023 20:19:11 +0800
Subject: [PATCH] FIX:frequent calls to _update_imgui_select_plate_toolbar

Jira: STUDIO-5488
Change-Id: I12e6f37c2fe94de004aa6da43421970d6df10f0f
---
 src/slic3r/GUI/GLCanvas3D.cpp |  3 ++-
 src/slic3r/GUI/IMToolbar.cpp  |  7 +++++++
 src/slic3r/GUI/IMToolbar.hpp  |  4 ++--
 src/slic3r/GUI/Plater.cpp     | 17 ++++++++++-------
 4 files changed, 21 insertions(+), 10 deletions(-)

diff --git a/src/slic3r/GUI/GLCanvas3D.cpp b/src/slic3r/GUI/GLCanvas3D.cpp
index 1a08ff827..ac4223144 100644
--- a/src/slic3r/GUI/GLCanvas3D.cpp
+++ b/src/slic3r/GUI/GLCanvas3D.cpp
@@ -6273,7 +6273,7 @@ void GLCanvas3D::_update_select_plate_toolbar_stats_item(bool force_selected) {
 bool GLCanvas3D::_update_imgui_select_plate_toolbar()
 {
     bool result = true;
-    if (!m_sel_plate_toolbar.is_enabled()) return false;
+    if (!m_sel_plate_toolbar.is_enabled() || m_sel_plate_toolbar.is_render_finish) return false;
 
     _update_select_plate_toolbar_stats_item();
 
@@ -7718,6 +7718,7 @@ void GLCanvas3D::_render_imgui_select_plate_toolbar()
     }
 
     imgui.end();
+    m_sel_plate_toolbar.is_render_finish = true;
 }
 
 //BBS: GUI refactor: GLToolbar adjust
diff --git a/src/slic3r/GUI/IMToolbar.cpp b/src/slic3r/GUI/IMToolbar.cpp
index e14cdb2ff..1dbfc9e61 100644
--- a/src/slic3r/GUI/IMToolbar.cpp
+++ b/src/slic3r/GUI/IMToolbar.cpp
@@ -58,6 +58,13 @@ void IMToolbar::del_stats_item()
     m_all_plates_stats_item = nullptr;
 }
 
+void IMToolbar::set_enabled(bool enable)
+{
+    m_enabled = enable;
+    if (!m_enabled)
+        is_render_finish = false;
+}
+
 bool IMReturnToolbar::init()
 {
     bool compress = false;
diff --git a/src/slic3r/GUI/IMToolbar.hpp b/src/slic3r/GUI/IMToolbar.hpp
index 681045fdb..add067d83 100644
--- a/src/slic3r/GUI/IMToolbar.hpp
+++ b/src/slic3r/GUI/IMToolbar.hpp
@@ -49,7 +49,7 @@ public:
     float icon_height;
     bool is_display_scrollbar;
     bool show_stats_item{ false };
-
+    bool is_render_finish{false};
     IMToolbar() {
         icon_width = DEFAULT_TOOLBAR_BUTTON_WIDTH;
         icon_height = DEFAULT_TOOLBAR_BUTTON_HEIGHT;
@@ -63,7 +63,7 @@ public:
     float fontScale;
 
     bool is_enabled() const { return m_enabled; }
-    void set_enabled(bool enable) { m_enabled = enable; }
+    void set_enabled(bool enable);
 
     void set_icon_size(float width, float height) {
         icon_width = width;
diff --git a/src/slic3r/GUI/Plater.cpp b/src/slic3r/GUI/Plater.cpp
index 8f7991618..8edfd5d2a 100644
--- a/src/slic3r/GUI/Plater.cpp
+++ b/src/slic3r/GUI/Plater.cpp
@@ -5778,15 +5778,18 @@ void Plater::priv::set_current_panel(wxPanel* panel, bool no_slice)
         };
 
     //BBS: add the collapse logic
-    if (panel == preview && q->only_gcode_mode()) {
-        this->sidebar->collapse(true);
-        preview->get_canvas3d()->enable_select_plate_toolbar(false);
-    }
-    else if (panel == preview && q->using_exported_file() && (q->m_valid_plates_count <= 1)) {
-        preview->get_canvas3d()->enable_select_plate_toolbar(false);
+    if (panel == preview) {
+        if (q->only_gcode_mode()) {
+            this->sidebar->collapse(true);
+            preview->get_canvas3d()->enable_select_plate_toolbar(false);
+        } else if (q->using_exported_file() && (q->m_valid_plates_count <= 1)) {
+            preview->get_canvas3d()->enable_select_plate_toolbar(false);
+        } else {
+            preview->get_canvas3d()->enable_select_plate_toolbar(true);
+        }
     }
     else {
-        preview->get_canvas3d()->enable_select_plate_toolbar(true);
+        preview->get_canvas3d()->enable_select_plate_toolbar(false);
     }
 
     if (current_panel == panel)