From def5bd6797666d7d962051629e17fdf6e0c07d71 Mon Sep 17 00:00:00 2001 From: YuSanka Date: Fri, 3 Dec 2021 09:38:50 +0100 Subject: [PATCH] Follow-up to https://github.com/prusa3d/PrusaSlicer/commit/7272b2b083448859ec63798326f7b6feead0d071 Comment is extended --- src/slic3r/GUI/MainFrame.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/slic3r/GUI/MainFrame.cpp b/src/slic3r/GUI/MainFrame.cpp index cca14e570..8a9702c40 100644 --- a/src/slic3r/GUI/MainFrame.cpp +++ b/src/slic3r/GUI/MainFrame.cpp @@ -320,7 +320,10 @@ static void add_tabs_as_menu(wxMenuBar* bar, MainFrame* main_frame, wxWindow* ba bar_parent->Bind(wxEVT_MENU_OPEN, [main_frame, bar, is_mainframe_menu](wxMenuEvent& event) { wxMenu* const menu = event.GetMenu(); if (!menu || menu->GetMenuItemCount() > 0) { - event.Skip(); // it is verry important to next pricessing of the wxEVT_UPDATE_UI by this menu + // If we are here it means that we open regular menu and not a tab used as a menu + event.Skip(); // event.Skip() is verry important to next processing of the wxEVT_UPDATE_UI by this menu items. + // If wxEVT_MENU_OPEN will not be pocessed in next event queue then MenuItems of this menu will never caught wxEVT_UPDATE_UI + // and, as a result, "check/radio value" will not be updated return; }