Thread: Context Menu & Dialog Box :: MFC

It crashes you say.....

Are you sure that IRD_CONTEXTMENU1 actually represents your resource as when VC++ creates a menu it would usually call this IDR_CONTEXTMENU1

I tried the following and it worked ok

Code:

void CTestKuphDlg::OnRButtonUp(UINT nFlags, CPoint point) 
{
	CPoint mPointCurrent;	
	CMenu mPopupMenu;
	mPopupMenu.LoadMenu(IRD_CONTEXTMENU1);
	ClientToScreen(&point);
	CMenu *pContextMenu = mPopupMenu.GetSubMenu(0);
	pContextMenu->TrackPopupMenu(TPM_LEFTALIGN | 
		TPM_LEFTBUTTON | TPM_RIGHTBUTTON | TPM_RETURNCMD | 
		TPM_NONOTIFY,point.x, point.y,GetActiveWindow(), NULL);	
	pContextMenu->DestroyMenu();
	CDialog::OnRButtonUp(nFlags, point);
}

Originally posted by kuphryn
Thanks.

I included the wrong ID when calling mPopupMenu.LoadMenu().

Hey, you used pContextMenu->DestroyMenu(). Is that a necessity?

Kuphryn

LOL....no ......On looking again, I guess if I was going to free the resource I should have done mPopupMenu.DestroyMenu();....

I was trying to free it here because with your code you were calling LoadMenu() in the message handler.....not the best way to go I guess

A far better way to go is to load the menu once somewhere else in the code (maybe on the WM_INITDIALOG) and store the CMenu in the dialog's class as a class member variable......

Then call DestroyMenu() when the dialog closes