All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cole Robinson <crobinso@redhat.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: qemu-devel@nongnu.org, Anthony Liguori <aliguori@amazon.com>,
	kraxel@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2 4/4] gtk: Add "Grab On Click" option
Date: Wed, 02 Apr 2014 09:49:02 -0400	[thread overview]
Message-ID: <533C154E.7040005@redhat.com> (raw)
In-Reply-To: <s5h61mru9sz.wl%tiwai@suse.de>

On 04/02/2014 09:42 AM, Takashi Iwai wrote:
> At Wed, 02 Apr 2014 09:28:54 -0400,
> Cole Robinson wrote:
>>
>> On 04/02/2014 08:32 AM, Takashi Iwai wrote:
>>> I simply like it better, you don't? :)
>>>
>>
>> In fact, relative mouse mode is a pain without this feature, you need to
>> manually initiate a grab with ctrl+alt+g before mouse movement will even work.
>> Compare to our sdl front end, or virt-viewer, vinagre, virt-manager, where
>> grab-on-click is the default (there isn't even an option to disable that
>> behavior).
>>
>> I don't know what the original intention of the code was, but I think this
>> behavior should be the default. Anthony, Gerd, thoughts?
> 
> I noticed later that the original gtk-ui behavior is better when
> vmmouse driver is available, i.e. the absolute mode is working.  With
> grab-on-click, you have to ungrab at each time you want to leave from
> VM window.
> 
> So I think we should leave the default behavior as is, since vmmouse
> driver is likely available in most cases.  But, still having this in
> the menu makes our lives easier in case vmmouse isn't available or
> doesn't work with QEMU.
> 

Or we just do what sdl.c does (and virt-viewer, and vinagre, and virt-manager):

absolute mode == grab on hover, ungrab when pointer leaves the window
relative mode == grab on click, require manual ungrab

- Cole

  reply	other threads:[~2014-04-02 13:49 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-02 12:32 [Qemu-devel] [PATCH v2 0/4] Fix relative pointer tracking on Gtk UI (v2) Takashi Iwai
2014-04-02 12:32 ` [Qemu-devel] [PATCH v2 1/4] gtk: Use gtk generic event signal instead of motion-notify-event Takashi Iwai
2014-04-02 13:25   ` Cole Robinson
2014-04-02 15:17   ` Gerd Hoffmann
2014-04-02 15:26     ` Takashi Iwai
2014-04-02 12:32 ` [Qemu-devel] [PATCH v2 2/4] gtk: Fix the relative pointer tracking mode Takashi Iwai
2014-04-02 13:26   ` Cole Robinson
2014-04-02 12:32 ` [Qemu-devel] [PATCH v2 3/4] gtk: Remember the last grabbed pointer position Takashi Iwai
2014-04-02 13:27   ` Cole Robinson
2014-04-02 12:32 ` [Qemu-devel] [PATCH v2 4/4] gtk: Add "Grab On Click" option Takashi Iwai
2014-04-02 13:28   ` Cole Robinson
2014-04-02 13:42     ` Takashi Iwai
2014-04-02 13:49       ` Cole Robinson [this message]
2014-04-02 14:39     ` Gerd Hoffmann
2014-04-02 15:07       ` [Qemu-devel] [PATCH] gtk: Grab pointer on click when in relative mode Cole Robinson
2014-04-02 15:19         ` Gerd Hoffmann
2014-04-02 15:25           ` Takashi Iwai
2014-04-02 20:18             ` Cole Robinson
2014-04-03  6:19               ` Takashi Iwai

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=533C154E.7040005@redhat.com \
    --to=crobinso@redhat.com \
    --cc=aliguori@amazon.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=tiwai@suse.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.