EnableMenuItem() does not make menu item gray???

[Wenrich]

Hi! I come across a strange thing: I try to gray a menu item, the codes as follow: CMenu *pMenu = GetMenu(); CMenu *pOP = pMenu->GetSubMenu(2); pOP->EnableMenuItem(ID_OP_START, MF_GRAYED | MF_BYCOMMAND); ****************************************************** The menu item with ID ID_OP_START is supposed to be grayed and GetMenuState() does return MF_GRAYED; but the item is actually not grayed and can still function as it does. Do I need to do something in order for this method to work? Thanks a lot!

[RaviBee]

Wenrich wrote: Do I need to do something in order for this method to work? What you're doing will work for a menu attached to a dialog in a dialog based app. However, if you're using MFC's doc/view framework (SDI/MDI), you need to put the enable/disable logic in the command's UpdateUI handler. /ravi Let's put "civil" back in "civilization" http://www.ravib.com ravib@ravib.com

[will1383]

Ravi Bhavnani wrote: What you're doing will work for a menu attached to a dialog in a dialog based app. However, if you're using MFC's doc/view framework (SDI/MDI), you need to put the enable/disable logic in the command's UpdateUI handler. This may be the case, but what if you need the change the menus from a different location other than in the UpdateUI handler? do you basically have to create a boolean to keep track of the state and then check it in the UpdateUI? Is this the only way? Thanks! Dan

[Roger Allen]

Another method would be to handle the WM_INITMENUPOPUP message and enable/disable the items in that handler. Roger Allen Sonork 100.10016 I have a terminal disease. Its called life!

[RaviBee]

groover4life wrote: create a boolean to keep track of the state and then check it in the UpdateUI? Yes, that follows the standard MVC model and would be the preferred way imho. /ravi Let's put "civil" back in "civilization" http://www.ravib.com ravib@ravib.com