Context menu does not appear at exact mouse position Topic is solved

[Nelson Joseph]

Hi All,

I am using the following code to create context menu. It works fine but the issue is, it does not appear at the mouse position. Instead, it appears 30 pixel ablove from the mouse's Y value.

Code: Select all

void MyFrame::OnContextMenu( wxContextMenuEvent& event )
{
	 wxPoint point = event.GetPosition();
	 // If from keyboard
	 if( point.x == -1 && point.y == -1 )
	 {
		 wxSize size = GetSize();
		 point.x = size.x / 2;
		 point.y = size.y / 2;
	 }
	 else
	 {
		 point = ScreenToClient( point );
	 }

	 wxMenu menu;
	 menu.Append( wxID_ANY, _T( "Item-1" ) );
	 menu.Append( wxID_ANY, _T( "Item-2" ) );
	 menu.Append( wxID_ANY, _T( "Item-3" ) );

	 PopupMenu( &menu, point.x, point.y );
}

If I use the same code in separate small application, it works fine.

I am having wxFrame within that wxSashWindow. The right side sash window contains openGL like window. The context menu does not appear while we right click at opnGL window.

I have attached sample image.
Any suggestions, welcome.

[catalin]

I'd suggest not to try to calculate and manually set the menu position when the position is known. Just calling PopupMenu with the default coordinates always worked correctly for me.

[Nelson Joseph]

Catalin:

Thanks for your quick reply.
I changed the code as you said, but I cant see any changes in output. Same thing is happening.

[Nelson Joseph]

I'd suggest not to try to calculate and manually set the menu position when the position is known. Just calling PopupMenu with the default coordinates always worked correctly for me.

If I remove my toolbar, then it works perfectly. What is the relation between toolbar and context menu? Any suggestion helpful. Thanks in advance.

[catalin]

If I remove my toolbar, then it works perfectly.

Aha! Then see ticket #13223

[Nelson Joseph]

Thanks Catalin. It works fine.

[Nelson Joseph]

If I remove my toolbar, then it works perfectly.

Aha! Then see ticket #13223

Catalin:

One more problem arise. I have made change according to your suggestion. But the context menu goes out of frame if the frame is in restore down.
I have attached the screen shot for your perusal.

Any suggestion...

[catalin]

What is "restore down" ?
I don't see it going out of the screen, so what is the problem with the menu going out of the frame? I don't think it is supposed to be contained in the frame's rectangle on any OS.

[Nelson Joseph]

I have modified the code of DoPopupMenu() on window.cpp as in wxWidgets 2.9

Code: Select all

menu->UpdateUI();
wxPoint point;
if ( x == wxDefaultCoord && y == wxDefaultCoord )
	{
		point = wxGetMousePosition();
	}
	else
	{
		point = ClientToScreen( wxPoint( x, y ) );
	}

Thanks.