From 13783d873391d8b5c34593f67fc105715eb8cf74 Mon Sep 17 00:00:00 2001 From: Artur Wieczorek Date: Thu, 7 Jan 2016 21:45:17 +0100 Subject: [PATCH] Fixed regression in wxMenuItem::Check introduced by dbd98a10. Internal check flag (via wxMenuItemBase::Check) has to be set as a first to avoid generating a spurious wxEVT_MENU when menu item is checked. --- src/gtk/menu.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gtk/menu.cpp b/src/gtk/menu.cpp index e3f6abd366..221e8b2098 100644 --- a/src/gtk/menu.cpp +++ b/src/gtk/menu.cpp @@ -693,14 +693,13 @@ void wxMenuItem::Check( bool check ) wxFALLTHROUGH; case wxITEM_CHECK: + wxMenuItemBase::Check( check ); gtk_check_menu_item_set_active( (GtkCheckMenuItem*)m_menuItem, (gint)check ); break; default: wxFAIL_MSG( wxT("can't check this item") ); } - - wxMenuItemBase::Check( check ); } void wxMenuItem::Enable( bool enable )