[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/2] ui/cocoa.m: move ungrab to ctrl-alt-g
From: |
Programmingkid |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/2] ui/cocoa.m: move ungrab to ctrl-alt-g |
Date: |
Thu, 2 Nov 2017 17:42:43 -0400 |
> On Nov 2, 2017, at 12:06 PM, BALATON Zoltan <address@hidden> wrote:
>
> On Thu, 2 Nov 2017, Programmingkid wrote:
>>> On Nov 2, 2017, at 5:10 AM, Peter Maydell <address@hidden> wrote:
>>>
>>> On 5 October 2017 at 15:55, John Arbuckle <address@hidden> wrote:
>>>> Currently the cocoa user interface relys on the user pushing control-alt
>>>> to ungrab the mouse. This is patch changes the key combination to
>>>> control-alt-g to be in line with the GTK user interface.
>>>>
>>>> signed-off-by: John Arbuckle <address@hidden>
>>>> ---
>>>
>>>> +
>>>> + // release the mouse grab
>>>> + case Q_KEY_CODE_G:
>>>> + [self ungrabMouse];
>>>> + break;
>>>> }
>>>
>>> Testing this I have found that it makes the grab key be
>>> "ctrl+alt+ the key labelled 'g'", even if in the
>>> OSX host keyboard mapping that key doesn't produce the
>>> letter 'g'. This is in contrast to for instance the menu
>>> accelerators which honour the host keyboard layout, and
>>> it's also not what the GTK UI does. So I think we need
>>> to fix that.
>>
>> I just realized that the cocoa interface does not consider the keyboard
>> layout. Switching from QWERTY to DVORK I still see the same keys outputting
>> the same characters in OpenBIOS. This is a separate patch but sometime to
>> take note.
>
> Is it the cocoa interface or OpenBIOS?
It was with the cocoa interface.
> In case you are using an emulated USB keyboard, the very simple driver in
> OpenBIOS only has a US layout so this may be the reason (see
> openbios/drivers/usbhid.c). I'm not sure about ADB keyboards but you may want
> to try a few combinations to identify where the problem is before looking for
> it in QEMU's cocoa interface.
It looks like there is no problem with sending keys to the guest. The cocoa
interface behaves as it should.