Menu items grayed out?

[Jesper Knudsen]

All items are grayed out, why?

void CMyFrameWnd::OnRButtonUp(UINT nFlags, CPoint p)
{
CMenu menu;
menu.CreatePopupMenu();
menu.AppendMenu( MF_STRING | MF_ENABLED , 700 , "Zoom All" );
menu.AppendMenu( MF_STRING | MF_ENABLED , 701 , "Properties" );
ClientToScreen(&p);
menu.TrackPopupMenu(TPM_CENTERALIGN,p.x,p.y,this,NULL);
CFrameWnd::OnRButtonUp(nFlags, point);
}

[David Crow]

Do you have a handler function for menu items 700 and 701?

---

"Take only what you need and leave the land as you found it." - Native American Proverb

[Jesper Knudsen]

No I am overriding OnCommand() For CWnd's this seems to be enough - maybe for CFrameWnd's it's not enough?

[David Crow]

MFC's framework will disable menu items for which a handler has not been provided. There is no way for it to know that you are handling menu item requests from within OnCommand().

---

"Take only what you need and leave the land as you found it." - Native American Proverb

[Jesper Knudsen]

So I shouldn't override OnCommand(), instead I should create a handler? ON_COMMAND_RANGE( 700 , 701 , OnMenuCommand ) afx_msg void OnMenuCommand ( UINT nID ) I just wonder - why is OnCommand() virtual then?

[John M Drescher]

You can add onupdatecmdui handlers for each id and then use oncmommand, I do this a lot. John

[Jesper Knudsen]

Thank you for your help, it works with the handler - still I wonder, still I wonder.. :)

[David Crow]

Jesper Knudsen wrote:

So I shouldn't override OnCommand(), instead I should create a handler?

That's how I would proceed.

Jesper Knudsen wrote:

I just wonder - why is OnCommand() virtual then?

It has its purpose. My most recent use of it was on a property sheet. I needed to handle the clicking of the OK button.

---

"Take only what you need and leave the land as you found it." - Native American Proverb

[David Crow]

MFC's message routing does take a little bit to get used to. The framework looks for an entry in the BEGIN_MESSAGE_MAP() method for both menu items and toolbar buttons. If one is not found, it does not enable that item or button. http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vclib/html/_mfcnotes_tn021.asp[^]

---

"Take only what you need and leave the land as you found it." - Native American Proverb