[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 6/8] ui/cocoa: Interpret left button down as is when command is pr
From: |
Gerd Hoffmann |
Subject: |
[PULL 6/8] ui/cocoa: Interpret left button down as is when command is pressed |
Date: |
Fri, 19 Feb 2021 23:00:34 +0100 |
From: Akihiko Odaki <akihiko.odaki@gmail.com>
Old Macs were not equipped with mice with an ability to perform
"right clicks" and ui/cocoa interpreted left button down with
left command key pressed as right button down as a workaround.
The workaround has an obvious downside: you cannot tell the guest
that the left button is down while the left command key is
pressed.
Today, Macs has trackpads, Apple Mice, or Magic Mice. They are
capable to emulate right clicks with gestures, which also allows
to perform right clicks on "BootCamp" OSes like Windows.
By removing the workaround, we overcome its downside, and provide
a behavior consistent with BootCamp.
Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
Message-Id: <20210212000706.28616-1-akihiko.odaki@gmail.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
ui/cocoa.m | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/ui/cocoa.m b/ui/cocoa.m
index eab4bfe7c8ae..13f19bece14d 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -835,11 +835,7 @@ QemuCocoaView *cocoaView;
mouse_event = true;
break;
case NSEventTypeLeftMouseDown:
- if ([event modifierFlags] & NSEventModifierFlagCommand) {
- buttons |= MOUSE_EVENT_RBUTTON;
- } else {
- buttons |= MOUSE_EVENT_LBUTTON;
- }
+ buttons |= MOUSE_EVENT_LBUTTON;
mouse_event = true;
break;
case NSEventTypeRightMouseDown:
@@ -851,11 +847,7 @@ QemuCocoaView *cocoaView;
mouse_event = true;
break;
case NSEventTypeLeftMouseDragged:
- if ([event modifierFlags] & NSEventModifierFlagCommand) {
- buttons |= MOUSE_EVENT_RBUTTON;
- } else {
- buttons |= MOUSE_EVENT_LBUTTON;
- }
+ buttons |= MOUSE_EVENT_LBUTTON;
mouse_event = true;
break;
case NSEventTypeRightMouseDragged:
--
2.29.2
- [PULL 0/8] Ui 20210219 patches, Gerd Hoffmann, 2021/02/19
- [PULL 3/8] ui/cocoa: Support unique keys of JIS keyboards, Gerd Hoffmann, 2021/02/19
- [PULL 4/8] ui/cocoa: Do not copy members of pixman image, Gerd Hoffmann, 2021/02/19
- [PULL 2/8] spice: flush drawing before notifying client, Gerd Hoffmann, 2021/02/19
- [PULL 7/8] ui/cocoa: Statically allocate dcl, Gerd Hoffmann, 2021/02/19
- [PULL 8/8] ui/console: Remove dpy_gl_ctx_get_current, Gerd Hoffmann, 2021/02/19
- [PULL 6/8] ui/cocoa: Interpret left button down as is when command is pressed,
Gerd Hoffmann <=
- [PULL 1/8] spice: flush on GL update before notifying client, Gerd Hoffmann, 2021/02/19
- [PULL 5/8] spice-app: avoid crash when core spice module doesn't loaded, Gerd Hoffmann, 2021/02/19
- Re: [PULL 0/8] Ui 20210219 patches, Peter Maydell, 2021/02/20