From dafc8c5f9223288fb22b8d45329b6621068c27ca Mon Sep 17 00:00:00 2001 From: Clifford Garwood Date: Fri, 9 Oct 2026 01:29:18 -0400 Subject: [PATCH] Let the pre-slice warnings checkbox redraw when clicked Its toggle handler did not call Skip(), so CheckBox's own handler, which redraws the tick, never ran. Each click still flipped and saved the setting, but the box kept showing it ticked. Co-Authored-By: Claude Opus 5.5 --- src/slic3r/GUI/Tab.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/slic3r/GUI/Tab.cpp b/src/slic3r/GUI/Tab.cpp index e90b46fc4d..6921bab6cf 100644 --- a/src/slic3r/GUI/Tab.cpp +++ b/src/slic3r/GUI/Tab.cpp @@ -5856,8 +5856,9 @@ if (is_marlin_flavor) auto* cb = new ::CheckBox(parent); const bool enabled = wxGetApp().app_config->get("imex_pre_slice_warnings") != "false"; cb->SetValue(enabled); - cb->Bind(wxEVT_TOGGLEBUTTON, [cb](wxCommandEvent&) { + cb->Bind(wxEVT_TOGGLEBUTTON, [cb](wxCommandEvent& e) { wxGetApp().app_config->set("imex_pre_slice_warnings", cb->GetValue() ? "true" : "false"); + e.Skip(); // CheckBox's own handler redraws the tick }); auto* s = new wxBoxSizer(wxHORIZONTAL); s->Add(cb, 0, wxALIGN_CENTER_VERTICAL);