Reference to a window control from context menu item click event handler

Gee Sam,

he is not doing anything that is inconsistent and even if he where, he has a reason for doing things the way he wants to.

" the easiest way would be to add a ContextMenu to the label(s) when the form is created" He already does that Sam.

If the code he has used as his solution works then that is fine, if it can be done better then he will find a way that is EVOLUTION.

There might be a solution that is more consistent with the design of Windows and .Net but I don't understand what you are trying to do. I assume that what you are trhing to do is very reasonable so please don't think I am criticizing anything but the following are things that seem inconsistent.
 
You say you "want to make the context menu I am using to be generic to all label controls" but you also say "I am trying to get a reference to the label control from the ToolStripMenuItem click event"; that seems inconsistent to me.
 
If I understand what you are trying to do, then the easiest way would be to add a ContextMenu to the label(s) when the form is created. I understand that you don't want to "iterate through all the controls on the form" but I am suggesting going the otehr direction; just process the labels in the ToolStripMenuItem or whatever.

Your solution is a better way good work.

Ok I have a better understanding, since you can only click on one label a time and the label's click event is fired before the context menu click, the reference control is already known so all you need to do is set which is the currentLabel in the click event then access the current label properties

It works for me in test, I hope this helps.

I am not sure if this can be improved but it achieves my goal.
        private void copyTextToClipboardToolStripMenuItem_Click(object sender, EventArgs e)
        {
            try
            {
                #region Try
                ToolStripMenuItem menuItem = (ToolStripMenuItem)sender;
                ToolStrip menu = menuItem.GetCurrentParent();
                ContextMenuStrip menuStrip = (ContextMenuStrip)menu;
                Label lbl = (Label)(menuStrip.SourceControl);
                SetClipboardText(lbl.Text);
                #endregion Try
            }
            catch (Exception)
            {
            }
        }

Who owns the menu?

What are you trying to do? Is it possible you may be going about it the wrong way?

These are my thoughts, you want to drop a context menu from a control, any control and show a value related to that control, is this about right?

If so then you do not need to know who sender is in respect to the way you are going about it now but you do need to know who wants / needs to show the menu.

In the MouseClick event (the same MouseClick event can be assigned to multiple objects)

    private void GENERIC_MouseClick(object sender, MouseEventArgs e)
        {
        switch(sender.getType().Name)
           {
         case "Label":
          //cast to the sender control type       
          mnuLabel = (Label)sender;
          mnuLabelText = mnuLabel.Text;
          }

        }


                SetClipboardText(mnuLabelText);
This is assuming I understand your Q correctly..