NSEvent click right menu

Question

I use this code to have two different behaviors when click right or left on menu.

Click left and click right + cmd.

How can I right click without pressing cmd + click in the simplest way?

-(void)awakeFromNib {

NSImage *image = [NSImage imageNamed:@"menubar"];
NSImage *alternateImage = [NSImage imageNamed:@"menubar-white"];

statusItem = [[NSStatusBar systemStatusBar] statusItemWithLength:NSVariableStatusItemLength];
[statusItem setHighlightMode:YES];
[statusItem setImage:image];
[statusItem setAlternateImage:alternateImage];
[statusItem setAction:@selector(show)];
}


- (void)show {

NSLog(@"call show");

NSEvent *event = [NSApp currentEvent];
//Respond to the mouse click
if ([event modifierFlags] & NSCommandKeyMask) //Command
{
    NSLog(@"RIGHT");
    [statusItem setMenu:statusMenu];
}
else {
    NSLog(@"LEFT");
    //open window
}
}

Thanks for your help!

Answer 1

I disagree with Aaron. In general, you should avoid inspecting the instantaneous mouse or keyboard state. It might have changed in the time since the action which you're actually supposed to respond to. For example, if the user left-clicks and then releases the mouse button, +pressedMouseButtons may return 0 by the time your code gets around to calling it.

Instead, you should examine the event that triggered the current processing. For a left-click, you would get an event whose type is NSLeftMouseDown. For a right-click, you would get NSRightMouseDown. If you already know that you have some kind of mouse click event and for some reason don't wish to examine its type, you can examine its buttonNumber property.

In fact, what has invoked your -show method? I would expect that you've implemented the NSResponder method -mouseDown: somewhere. If so, that corresponds to a left-click. If the user right-clicks, a different method (-rightMouseDown:) gets called. So, you should generally code those two methods differently if you want a different response.

Answer 2

Inspect [NSEvent pressedMouseButtons] instead of the modifier flags. Let the system take care of deciding which button has been clicked. If you do it the way you're trying to do now you'll get weird behavior for users who are actually using multi-button mice.

You should be able to use something like this:

const NSUInteger pressedButtonMask = [NSEvent pressedMouseButtons];
const BOOL leftMouseDown = (pressedButtonMask & (1 << 0)) != 0;
const BOOL rightMouseDown = (pressedButtonMask & (1 << 1)) != 0;