Get click event on a menuitem with subitems (C#)

Question

I am creating a contextmenu that should contain a listing of all folders, subfolders, and files in a chosen folder. I want the menu to open the files and folders when they are clicked, but the click event doesn't register if the menuitem has subitems.

void Foo(string Title)
{
    MenuItem = new MenuItem(Title);
    MenuItem.Click += new EventHandler(MenuItem_Click);
    ContextMenu.MenuItems.Add(MenuItem);
}

void MenuItem_Click(object sender, EventArgs e)
{
    MessageBox.Show("This box will only show when menuitems without subitems are clicked");
}

How can I make the click event fire even if the menuitem has subitems?

Answer 1

It sounds like a menu may not be the most appropriate UI widget here. I believe the reason you don't get a click event raised is that menu items with submenus are assumed to only expand their children when clicked, rather than executing any other action.

That's likely to be the assumption of the user, too.

This behaviour is mentioned in the documentation for MenuItem.Click:

Note: If the MenuItems property for the MenuItem contains any items, this event is not raised. This event is not raised for parent menu items.

Answer 2

If you launched your popup from a toolstrip, you can subclass the toolstrip and add this code.

  override protected void OnItemClicked(ToolStripItemClickedEventArgs e)
  {
    if (this.Items.Count == 0)
      base.OnItemClicked(e);

    // else do nothing
  }

However, the ContextMenu class does not have OnItemClicked, but it has onPopup. I have not tried it but you could try subclassing the ContextMenu

public class MyContextMenu : ContextMenu
{
  override protected void OnPopUp(EventArgs e)
  {
        if (this.MenuItems.Count == 0)
          base.OnPopUp(e);

        // else do nothing
  }
}

if that does not work then, you could override the two show methods

public class MyContextMenu : ContextMenu
{
  override protected void Show (Control c, Point p)
  {
        if (this.MenuItems.Count == 0)
          base.Show (c, p) ;

        // else do nothing
  }

  override protected void (Control c, Point p, LeftRightAlignment z) 
  {
        if (this.MenuItems.Count == 0)
          base.Show (c, p, z) ;

        // else do nothing
  }
}

Answer 3

I had the same issue, the solution was to use the MenuItem.SubmenuOpened event instead of the click event.