[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] gtk: Fix accelerator filtering
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH v2] gtk: Fix accelerator filtering |
Date: |
Mon, 25 Feb 2013 16:44:39 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2013-02-25 16:39, Anthony Liguori wrote:
> Jan Kiszka <address@hidden> writes:
>
>> This is in fact very simply: When the input in grabbed, everything
>> should be exclusively passed to the guest - except it has our magic
>> CTRL-ALT modifier set. Then let GTK filter out those accels that are in
>> use. When checking the modifier state, we just need to filter out NUM
>> and CAPS lock.
>
> Can you explain what you're fixing?
That it's not filtering what it is supposed to.
>
> We shouldn't hard code modifiers like this. The reason you give
> accelerators paths like this is so that they can be overridden by a
> user.
>
> That's why I filtered by path. Once we're running, we shouldn't assume
> that accelerators use the modifiers we started with.
Your path-based filtering does not work as it uses an unsupported
internal function (see my other mail).
We can make the modifier configurable via QEMU means (command line
parameter, gconfig, whatever). But let's get the basics working first.
Jan
--
Siemens AG, Corporate Technology, CT RTC ITP SDP-DE
Corporate Competence Center Embedded Linux
[Qemu-devel] [PATCH 8/9] gtk: make default UI (v5), Anthony Liguori, 2013/02/20
[Qemu-devel] [PATCH 9/9] gtk: suppress accelerators from the File menu when grab is active, Anthony Liguori, 2013/02/20
Re: [Qemu-devel] [PATCH 0/9] Add GTK UI to enable basic accessibility (v5), Andreas Färber, 2013/02/20
Re: [Qemu-devel] [PATCH 0/9] Add GTK UI to enable basic accessibility (v5), Anthony Liguori, 2013/02/22