From: BALATON Zoltan <balaton@eik.bme.hu>
To: Programmingkid <programmingkidx@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH v2 1/2] ui/cocoa.m: move ungrab to ctrl-alt-g
Date: Thu, 2 Nov 2017 17:06:06 +0100 (CET) [thread overview]
Message-ID: <alpine.BSF.2.21.1711021700510.67752@zero.eik.bme.hu> (raw)
In-Reply-To: <385E45E2-3BC6-4501-91B1-44CA43718947@gmail.com>
On Thu, 2 Nov 2017, Programmingkid wrote:
>> On Nov 2, 2017, at 5:10 AM, Peter Maydell <peter.maydell@linaro.org> wrote:
>>
>> On 5 October 2017 at 15:55, John Arbuckle <programmingkidx@gmail.com> 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 <programmingkidx@gmail.com>
>>> ---
>>
>>> +
>>> + // 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? 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.
Regards,
BALATON Zoltan
next prev parent reply other threads:[~2017-11-02 16:06 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-05 14:55 [Qemu-devel] [PATCH v2 0/2] ui/cocoa.m: enable guest to see control-alt key combinations John Arbuckle
2017-10-05 14:55 ` [Qemu-devel] [PATCH v2 1/2] ui/cocoa.m: move ungrab to ctrl-alt-g John Arbuckle
2017-11-02 9:10 ` Peter Maydell
2017-11-02 15:20 ` Programmingkid
2017-11-02 16:06 ` BALATON Zoltan [this message]
2017-11-02 21:42 ` Programmingkid
2017-11-02 16:09 ` Peter Maydell
2017-10-05 14:55 ` [Qemu-devel] [PATCH v2 2/2] ui/cocoa.m: send ctrl-alt key combinations to guest if not used by QEMU John Arbuckle
2017-10-31 14:38 ` Peter Maydell
2017-10-31 14:52 ` Peter Maydell
2017-11-01 15:11 ` Programmingkid
2017-10-05 16:39 ` [Qemu-devel] [PATCH v2 0/2] ui/cocoa.m: enable guest to see control-alt key combinations no-reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.BSF.2.21.1711021700510.67752@zero.eik.bme.hu \
--to=balaton@eik.bme.hu \
--cc=peter.maydell@linaro.org \
--cc=programmingkidx@gmail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).