CODEWITHSUNDEEP
c#comboboxwinformsright-click
marco0009
marco0009

Reputation: 163

Unable to detect right mouseclick in ComboBox

I have a ComboBox that is a simple drop down style. I wanted to open a new window when the user right clicks on an item in the list, but am having trouble getting it to detect a right click has occurred.

My code:

private void cmbCardList_MouseClick(object sender, MouseEventArgs e)
{
    if (e.Button == MouseButtons.Right && cmbCardList.SelectedIndex != -1)
    {
        frmViewCard vc = new frmViewCard();
        vc.updateCardDisplay(cmbCardList.SelectedItem);
        vc.Show();
    }
}

If I change e.Button == MouseButtons.Left the whole thing fires off just fine. Any way I can get this working as I intend?

Upvotes: 6

Views: 7652

Answers (4)

Tomek
Tomek

Reputation: 69

I used the MouseDown in the very simplest way:

        if (e.Button != MouseButtons.Right) return;
        (new Controls.FormComboContextMenu()).Show(cmb, cmb.PointToClient(Cursor.Position + new Size(5, 5)));

And it works all the time no matter we click all around with both mouse buttons.

Upvotes: 0

IlPADlI
IlPADlI

Reputation: 2033

You can use the Opening event of ContextMenuStrip to handle right click event.

var chk = new CheckBox();
chk.ContextMenuStrip = cmsNone;

private void cmsNone_Opening(object sender, CancelEventArgs e)
{
    e.Cancel = true;
    var cms = (ContextMenuStrip)sender;
    var chk = cms.SourceControl;
    //do your stuff
}

Upvotes: 0

KeithS
KeithS

Reputation: 71563

As an epitaph to this question, you can make this work using normal .NET functionality; you just have to go a little deeper into the event call stack. Instead of handling the MouseClick event, handle the MouseDown event. I had to do something similar recently, and I simply overrode the OnMouseDown method instead of attaching a handler. But, a handler should work too. Here's the code:

    protected override void OnMouseDown(MouseEventArgs e)
    {
        if (e.Button == MouseButtons.Right && !HandlingRightClick)
        {
            HandlingRightClick = true;
            if (!cmsRightClickMenu.Visible)
                cmsRightClickMenu.Show(this, e.Location);
            else cmsRightClickMenu.Hide();
        }
        base.OnMouseDown(e);
    }

    protected override void OnMouseUp(MouseEventArgs e)
    {
        HandlingRightClick = false;
        base.OnMouseUp(e);
    }

    private bool HandlingRightClick { get; set; }

The HandlingRightClick property is to prevent multiple triggers of the OnMouseDown logic; the UI will send multiple MouseDown messages, which can interfere with hiding the right-click menu. To prevent this, I only perform the logic once on the first MouseDown trigger (the logic's simple enough that I don't care if two invocations happen to race, but you might), then ignore any other MouseDown triggers until a MouseUp occurs. It's not perfect, but this'll do what you need it to.

Upvotes: 6

o.k.w
o.k.w

Reputation: 25810

I'm afraid that will not be posible unless you do some serious hacking. This article will explain.

Quoted for you:

Individual Controls

The following controls do not conform to the standard mouse click event behavior:

Button, CheckBox, ComboBox, and RadioButton controls

  • Left click: Click, MouseClick

  • Right click: No click events raised

  • Left double-click: Click, MouseClick; Click, MouseClick

  • Right double-click: No click events raised

Upvotes: 9

Related Questions