C# Button Click, Hold and Release

Asterix Obelix

Hi,

I need to do a button which has same functionality as PTT (Press to Talk).
So, you press and keep the button pressed - it does some function.
On release of the button, do something else.

while (button.pressed)
do something

on release
do something

ps. I found this code in a Forum, but it is not working. Nothing happens on pressing the button.

isLooping = false;

//on mouse down
private void myControl_MouseUp(object sender, System.Windows.Forms.MouseEventArgs e) {
    isLooping = true;
    runLoop();
}

//on mouse up event
private void myControl_MouseUp(object sender, System.Windows.Forms.MouseEventArgs e) {
    isLooping = false;
}

//This is the main loop you care about.  Put this in your application
//This should go in its own thread
void runLoop() {
    while (isLooping) {
        //do stuff
    }
}

14 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted answer
As we saw in your first question, you always need to wire up eventhandlers to their events, otherwise they simply won't fire.

This is done automatically for you by VS if you add a skeleton handler by double clicking on the event in the Properties Box, otherwise you have to do it yourself.

If you've just copied and pasted my suggested code for the 'hotkey', then you'll need to add these lines to your Form_Load eventhandler:

this.KeyDown += Form1_KeyDown;
this.KeyUp += Form1_KeyUp;

The code definitely works as I tried it myself before posting.
0
Ganeshan R9y ago
how to create a mute program in c3
0
Asterix Obelix15y ago
@"Vulpes", Thank you for your great explanation. It works and I apologize for asking another question in the same thread!
0
Sam Hobbs15y ago
You should ask new questions in a new thread. There are many reasons for that. In other words, the original question asked about a PTT button and you got that question answered. Then you asked about a hotkey; that is a new question.

One of the many reasons for making a new thread is that the question about a hotkey is very unclear to me. I am very confused about what that actually means, but I won't even try to get clarification in this thread.
0
Asterix Obelix15y ago
Hi,

I tried what you suggested for the Hotkey, but it is not working.
Maybe I need also to link the methods (events) that you suggested as in the previous case?

I thought of putting the following link on form_load, but a detail is missing since it is not working:

this.button2.KeyDown += new KeyEventHandler(Form1_KeyDown);
this.button2.KeyUp += new KeyEventHandler(Form1_KeyUp);

0
Vulpes15y ago
An easy way to do this without installing a keyboard hook is to first set the form's KeyPreview property to true and then to handle its KeyDown and KeyUp events on the following lines. Here, I'm using Ctrl+Alt+Z as the hotkey:

private void Form1_KeyDown(object sender, KeyEventArgs e)
{
if (e.KeyCode == Keys.Z && e.Modifiers == (Keys.Control | Keys.Alt))
{
e.Handled = true;
isLooping = true;
runLoop();
}
}

private void Form1_KeyUp(object sender, KeyEventArgs e)
{
if (e.KeyCode == Keys.Z && e.Modifiers == (Keys.Control | Keys.Alt))
{
e.Handled = true;
isLooping = false;
}
}
0
Asterix Obelix15y ago
Thank you! That was the issue why it didn't work.

I would like to ask another think.

How can I link a Hotkey to do this functionality of pressing the Button?

Thank you again!
0
Vulpes15y ago
Are you sure that your original MouseDown and MouseUp handlers are wired up to the corresponding events i.e. you've added them from the Properties Box rather than just copying and pasting the code into your project?

If they're not wired up, then you can still do it by adding code such as this to the Form_Load handler:

// use actual button name, of course, in place of myControl

myControl.MouseUp += myControl_MouseUp;
myControl.MouseDown += myControl_MouseDown;
0
Asterix Obelix15y ago
I tried to do what you suggested - on MouseDown and MouseUp put the related skype commands.
But still, what should I add on the button itself (button click)?

Here is the code:

private void button1_Click_1(object sender, EventArgs e)
{

}

private void OnMouseDown(object sender, System.Windows.Forms.MouseEventArgs e)
{
Skype.Attach(6, true);
SKYPE4COMLib.Command command1 = new SKYPE4COMLib.Command();
command1.Command = "SET MUTE OFF";
command1.Blocking = false;
command1.Id = 1000;
Skype.SendCommand(command1);
}

//on mouse up event
private void OnMouseUp(object sender, System.Windows.Forms.MouseEventArgs e)
{
Skype.Attach(6, true);
SKYPE4COMLib.Command command1 = new SKYPE4COMLib.Command();
command1.Command = "SET MUTE ON";
command1.Blocking = false;
command1.Id = 1000;
Skype.SendCommand(command1);

}
0
Jaganathan Bantheswaran15y ago
Then simply remove the loop and on the micro phone in pressed state off that on released state
0
Asterix Obelix15y ago
I don't know whether the purpose why I am using runLoop is right. Here below is the code for turning the microphone on Skype On/Off
__________________________________

void runLoop2() {
Skype.Attach(6, true);
SKYPE4COMLib.Command command1 = new SKYPE4COMLib.Command();
command1.Command = "SET MUTE ON";
command1.Blocking = false; command1.Id = 1000;
Skype.SendCommand(command1);
}
_______________________________

So, my idea is to mute the microphone on Pressed and on released unmute.
0
Jaganathan Bantheswaran15y ago
Hi,

Put break point on both events and the loop. check whether the isLooing is getting changed while press and release.
0
Asterix Obelix15y ago
Actually, I changed the first one. Also, the runloop() I have specific function that needs to run, but still is not working!
0
Jaganathan Bantheswaran15y ago
Hi,

I cant find out any MouseDown event handler in that code.... Change the first _MouseUp event handler to _MouseDown.
0