Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

User control click event not working when clicking on text inside control?

I have a user control called GameButton that has a label inside it. When I add the user control to my form, and add a click event to it, its triggered when you click on the background of the custom button, but not the text in the label? How would I fix this without adding a bunch of click events inside the user controls code?

edit: UI framework: winforms

like image 796
Oztaco Avatar asked Mar 13 '12 01:03

Oztaco


4 Answers

If I am understanding you properly, your GameButton usercontrol will fire the event when clicked on, but not when the label is clicked on -- and you want both. This is because the label (a control) is on top of the background. Therefore, you need to register your label with the click event as well. This can be done manually in the designer or programmatically for each control on the page.

If you want to do EVERY control in the UserControl, put this into the UserControl's OnLoad event and you can use the same click event for every control:

foreach (var c in this.Controls)
    c.Click += new EventHandler(yourEvent_handler_click);

public void yourEvent_handler_click (object sender, EventArgs e){
    //whatever you want your event handler to do
}

EDIT: The best way is to create the click event handler property in the user control. This way, every time you add/remove a click event to your user control, it adds/removes it to all the controls within the user control automatically.

public new event EventHandler Click {
        add {
            base.Click += value;
            foreach (Control control in Controls) {
                control.Click += value;
            }
        }
        remove {
            base.Click -= value;
            foreach (Control control in Controls) {
                control.Click -= value;
            }
        }
    }

This is as per another post:

Hope this helps!

like image 55
ImGreg Avatar answered Nov 03 '22 16:11

ImGreg


You can create a new method and assign all the controls to it

private void Control_Click(object sender, EventArgs e)
{
   this.OnClick(e);
}

This will raise main control(or usercontrol) event.

like image 20
Shervin Ivari Avatar answered Nov 03 '22 18:11

Shervin Ivari


Set the "enable" property of your labels "False, then mouse events will work in user control.

like image 38
Amin Mir Avatar answered Nov 03 '22 16:11

Amin Mir


You can make the events in the controls of the User Control call the event of the User Control like that:

foreach (Control c in this.Controls)
{
    c.Click += (sender, e) => { this.OnClick(e); };
    c.MouseUp += (sender, e) => { this.OnMouseUp(e); };
    c.MouseDown += (sender, e) => { this.OnMouseDown(e); };
    c.MouseMove+= (sender, e) => { this.OnMouseMove(e); };
}

Just put it in the constructor. This way when an event is added to the User Control using polymorphism it will work

like image 1
davi133 Avatar answered Nov 03 '22 18:11

davi133