Detect any of the control pressing

Question

I am designing a small application in which I open a new form with some controls. I want to close this form if idle for some time. Let say the form is having many buttons, scroll bars, picture box....if none of them is clicked (within 10 seconds) the form should close.

I have used a timer of 10 seconds , i want to reset it if any of the control is pressed. I can do it individually but would become very cumbersome doing it for all the controls. Can i do it by some simpler way....I read on net:

foreach (Control cc in Controls)

but didn't knew how to use it. Please help.

isnt there a way to make an method that you add to all the onclick eventhandles? I think there is no shorter (less code) way than this. — Joel Harkes, Dec 15 '16 at 07:36
what sort of application is this? Winforms, WPF, ASP.Net Web Forms or MVC? — Simon Price, Dec 15 '16 at 07:36

Pepernoot · Answer 1 · 2016-12-15T07:46:27.167

0

This should get you started

    private void Form1_Load(object sender, EventArgs e)
    {
        foreach(Control control in this.Controls)
        {
            control.Click += Oncontrol_Click;
        }
    }

    private void Oncontrol_Click(object sender, EventArgs e)
    {
        Control control = sender as Control;
        MessageBox.Show($"{control.Text} is clicked");
    }

edited Dec 15 '16 at 07:46

answered Dec 15 '16 at 07:41

Pepernoot

3,409
3
21
46

score 0 · Answer 2 · answered Dec 15 '16 at 07:47

0

You can bind the click event to all the controls inside the foreach loop.

foreach (Control c in Controls)
{
    c.Click += (o, args) => ResetTimer();
}

Now each time a user clicks a control, ResetTimer() gets fired.

answered Dec 15 '16 at 07:47

abdul

1,562
1
17
22

score 0 · Answer 3 · answered Dec 15 '16 at 08:07

0

You need to do this to add the click event to all of the controls on a form:

    private void Form1_Load(object sender, EventArgs e)
    {
        AttachHandler(this, (s, e2) => ResetTimer());
    }

    private void AttachHandler(Control control, EventHandler handler)
    {
        control.Click += handler;
        foreach (Control c in control.Controls)
        {
            AttachHandler(c, handler);
        }
    }

It needs to recursively go down all controls that can contain other controls.

answered Dec 15 '16 at 08:07

Enigmativity

113,464
11
89
172

Thanks for your reply. The solution was helpful but if there are some controls inside a panel on the form, they don't reflect, should they be handled seperately – Amit Garg Dec 15 '16 at 11:22
@AmitGarg - What do you mean they "don't reflect"? This code should attach the handler for the controls on panels too. – Enigmativity Dec 15 '16 at 22:14
Sorry for confusion. The code is working properly for buttons , but is not working for scroll bars (horizontal scroll bar / vertical scroll bar / textbox scroll bar ). Can you please advise – Amit Garg Dec 16 '16 at 04:18
@AmitGarg - Scroll bars don't respond to clicks because they are there to scroll. You'll need to expand your code to handle scrolling controls and the scrolling events. – Enigmativity Dec 16 '16 at 09:51

Detect any of the control pressing

3 Answers3