From ff1e48a9c4ebf4a35d81351c1702797360ba3322 Mon Sep 17 00:00:00 2001 From: "xin.zhang" Date: Thu, 25 Sep 2025 11:07:45 +0800 Subject: [PATCH] FIX: the event should be skipped, so that the CheckBox could be toggled JIRA: [STUDIO-14861] Change-Id: I72e9e77c0e81e9cf64de7258ca09c225d5c3f807 (cherry picked from commit 9861c3d6bcd7e2720d1fccaacb820e13b5d7adb7) --- src/slic3r/GUI/CreatePresetsDialog.cpp | 6 ++++++ src/slic3r/GUI/FilamentMapDialog.cpp | 2 ++ src/slic3r/GUI/SyncAmsInfoDialog.cpp | 2 ++ 3 files changed, 10 insertions(+) diff --git a/src/slic3r/GUI/CreatePresetsDialog.cpp b/src/slic3r/GUI/CreatePresetsDialog.cpp index 80861a083c..849b6f6faa 100644 --- a/src/slic3r/GUI/CreatePresetsDialog.cpp +++ b/src/slic3r/GUI/CreatePresetsDialog.cpp @@ -838,6 +838,8 @@ wxBoxSizer *CreateFilamentPresetDialog::create_vendor_item() Refresh(); Layout(); Fit(); + + e.Skip(); }); comboBoxSizer->Add(vendor_sizer, 0, wxEXPAND | wxTOP, FromDIP(5)); @@ -1842,6 +1844,8 @@ wxBoxSizer *CreatePrinterPresetDialog::create_printer_item(wxWindow *parent) Layout(); m_page1->SetSizerAndFit(m_page1_sizer); Fit(); + + e.Skip(); }); vertical_sizer->Add(checkbox_sizer, 0, wxEXPAND | wxTOP, FromDIP(5)); @@ -1920,6 +1924,8 @@ wxBoxSizer *CreatePrinterPresetDialog::create_nozzle_diameter_item(wxWindow *par Layout(); m_page1->SetSizerAndFit(m_page1_sizer); Fit(); + + e.Skip(); }); vertical_sizer->Add(checkbox_sizer, 0, wxEXPAND | wxTOP, FromDIP(5)); diff --git a/src/slic3r/GUI/FilamentMapDialog.cpp b/src/slic3r/GUI/FilamentMapDialog.cpp index 3406255a86..1d246d1cda 100644 --- a/src/slic3r/GUI/FilamentMapDialog.cpp +++ b/src/slic3r/GUI/FilamentMapDialog.cpp @@ -274,6 +274,8 @@ void FilamentMapDialog::on_checkbox(wxCommandEvent &event) dialog.ShowModal(); this->Close(); } + + event.Skip(); } void FilamentMapDialog::on_ok(wxCommandEvent &event) diff --git a/src/slic3r/GUI/SyncAmsInfoDialog.cpp b/src/slic3r/GUI/SyncAmsInfoDialog.cpp index 56f7782bfc..7a7a65ca65 100644 --- a/src/slic3r/GUI/SyncAmsInfoDialog.cpp +++ b/src/slic3r/GUI/SyncAmsInfoDialog.cpp @@ -933,6 +933,7 @@ SyncAmsInfoDialog::SyncAmsInfoDialog(wxWindow *parent, SyncInfo &info) : auto flag = wxGetApp().app_config->get_bool("enable_append_color_by_sync_ams"); wxGetApp().app_config->set_bool("enable_append_color_by_sync_ams",!flag); m_append_color_checkbox->SetValue(!flag); + e.Skip(); }); m_append_color_checkbox->Hide(); m_append_color_sizer->Add(m_append_color_checkbox, 0, wxALIGN_LEFT | wxTOP, FromDIP(4)); @@ -953,6 +954,7 @@ SyncAmsInfoDialog::SyncAmsInfoDialog(wxWindow *parent, SyncInfo &info) : auto flag = wxGetApp().app_config->get_bool("enable_merge_color_by_sync_ams"); wxGetApp().app_config->set_bool("enable_merge_color_by_sync_ams",!flag); m_merge_color_checkbox->SetValue(!flag); + e.Skip(); }); m_merge_color_checkbox->Hide(); m_merge_color_sizer->Add(m_merge_color_checkbox, 0, wxALIGN_LEFT | wxTOP, FromDIP(2));