From 030f50ef4af13a1a13f476e22e60616a3d20d75f Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sun, 28 Aug 2005 13:17:29 +0000 Subject: [PATCH] modified GetLastMenuItem() to also take into account the popup menu items git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@35338 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- samples/menu/menu.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/samples/menu/menu.cpp b/samples/menu/menu.cpp index e147f5c5a1..5d1c12e335 100644 --- a/samples/menu/menu.cpp +++ b/samples/menu/menu.cpp @@ -547,11 +547,13 @@ wxMenuItem *MyFrame::GetLastMenuItem() const void MyFrame::LogMenuEvent(const wxCommandEvent& event) { int id = event.GetId(); - if ( !GetMenuBar()->FindItem(id) ) - return; wxString msg = wxString::Format(_T("Menu command %d"), id); - if ( GetMenuBar()->FindItem(id)->IsCheckable() ) + + // catch all checkable menubar items and also the check item from the popup + // menu + wxMenuItem *item = GetMenuBar()->FindItem(id); + if ( (item && item->IsCheckable()) || id == Menu_Popup_ToBeChecked ) { msg += wxString::Format(_T(" (the item is currently %schecked)"), event.IsChecked() ? _T("") : _T("not "));