Hi,
I'm trying to open a popup menu using mouse right click.
I'm using the following code to do so but the menu returnes as null.
The code:
CMenu menu;
menu.LoadMenu(IDR_MENU);
CMenu *pPopup = menu.GetSubMenu(0);
ASSERT(pPopup);

pPopup->TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, 20, 20, AfxGetMainWnd());

"pPopup" always returnes as null.
Any explanations?
Thanks,
Gabi

try changing this:

pPopup->TrackPopupMenu(TPM_LEFTALIGN | TPM_RIGHTBUTTON, 20, 20,this);

__________________
When you say "I wrote a program that crashed Windows," people just stare at you blankly and say "Hey, I got those with the system, for free." Linus Torvalds

Thanks for the reply.
My problem occurs earlier. I get the null after the line:
CMenu *pPopup = menu.GetSubMenu(0);

Gabi

well then the code is fine in my view, the problem might be when u create the menu in the resource, check this guidline when creating the menu:

__________________
When you say "I wrote a program that crashed Windows," people just stare at you blankly and say "Hey, I got those with the system, for free." Linus Torvalds

What do you mean by: "add all of the menu items in the first drop down on the menu"?
If I have the following menu Items: "one", "two", "three" and so on.
Do I put "one" at the head of the first drop down, Define it as popup menu (which means it does not have id) and put "two", "three" under it as non popup menus?
Thanks,
Gabi

Here is a picture shows how to do it:

Attached Images

menuPic.jpg (29.0 KB, 116 views)

__________________
When you say "I wrote a program that crashed Windows," people just stare at you blankly and say "Hey, I got those with the system, for free." Linus Torvalds

Nope.
You should put something else at the top of the menu - for example "Numbers" - this will be with the "Pop-up" option selected, and put "one", "two", "three" in this drop-down == "add all of the menu items in the first drop down on the menu"

It probably returns NULL because "one" is a normal item inside the popup.
Suppose you set the ID of the "one" item to IDC_ONE:

If you get the message box, it means you did it wrong 8-) (the code checks if you set the first item to non-poup).

Regards,
Luci

__________________
Please read these Guidelines before posting on the forum

"A person who never made a mistake never tried anything new."
Einstein

Oooups, sorry Max Payne, when I began replying, I didn't see your post (the page didn't refresh).

The picture is realy worth a thousand words

__________________
Please read these Guidelines before posting on the forum

"A person who never made a mistake never tried anything new."
Einstein

Thanks Max Payne and LucyWiz,
This solves my problem.
Gabi