wxString helpString;
int menuId = event.GetMenuId();
- if ( menuId != -1 )
+ if ( menuId != wxID_SEPARATOR && menuId != -2 /* wxID_TITLE */ )
{
wxMenuBar *menuBar = GetMenuBar();
- if (menuBar)
+ if ( menuBar )
{
- helpString = menuBar->GetHelpString(menuId);
+ // it's ok if we don't find the item because it might belong to
+ // the popup menu
+ wxMenuItem *item = menuBar->FindItem(menuId);
+ if ( item )
+ helpString = item->GetHelp();
}
}
wxMenuBar *bar = GetMenuBar();
if (!bar) return;
- wxMenuItem *item = bar->FindItemForId(id) ;
+ wxMenuItem *item = bar->FindItem(id) ;
if (item && item->IsCheckable())
{
- bar->Check(id,!bar->Checked(id)) ;
+ bar->Check(id, !bar->IsChecked(id)) ;
}
wxEvtHandler* evtHandler = GetEventHandler();