From: Takashi Iwai <tiwai@suse.de>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Cole Robinson <crobinso@redhat.com>,
Gerd Hoffmann <kraxel@redhat.com>,
Anthony Liguori <aliguori@amazon.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v3 4/4] gtk: Add "Grab On Click" option
Date: Tue, 08 Apr 2014 11:23:02 +0200 [thread overview]
Message-ID: <s5hbnwcxjih.wl%tiwai@suse.de> (raw)
In-Reply-To: <20140408085719.GC4880@redhat.com>
At Tue, 8 Apr 2014 11:57:19 +0300,
Michael S. Tsirkin wrote:
>
> On Tue, Apr 08, 2014 at 10:49:57AM +0200, Gerd Hoffmann wrote:
> > On Di, 2014-04-08 at 11:11 +0300, Michael S. Tsirkin wrote:
> > > On Tue, Apr 08, 2014 at 08:25:34AM +0200, Gerd Hoffmann wrote:
> > > > On Mo, 2014-04-07 at 23:38 +0300, Michael S. Tsirkin wrote:
> > > > > On Mon, Apr 07, 2014 at 10:07:43AM +0200, Gerd Hoffmann wrote:
> > > > > > On Fr, 2014-04-04 at 12:41 +0200, Takashi Iwai wrote:
> > > > > > > I simply like it better, you don't? :)
> > > > > >
> > > > > > I still think we should make this simply depend on absolute/relative
> > > > > > pointer mode instead of asking the user to switch it manually.
> > > > >
> > > > > Hmm how do I set absolute/relative mode?
> > > >
> > > > Depends on the pointer input device being used.
> > > >
> > > > Start guest with "-device usb-tablet". Go to HMP. 'info mice' should
> > > > list the ps/2 mouse and the usb tablet. Using 'mouse_set' (or was it
> > > > set_mouse?) you can force one of the two devides being used.
> > > >
> > > > If you pick the tablet (should be active by default) absolute pointer
> > > > events are passed to the guest, and qemu works in absolute mode (i.e.
> > > > sdl doesn't do pointer grabs).
> > > >
> > > > If you pick the mouse relative mouse events are passed to the guest, and
> > > > qemu works in relative mode (pointer grab is pretty much required to
> > > > work with the guest).
> > > >
> > > > cheers,
> > > > Gerd
> > > >
> > >
> > > I have to say grab on click is easier to understand - more predictable.
> > > Not requesting that it's made the default, but an option would be nice.
> >
> > Well, usually you'll never ever pick the pointer device manually, this
> > is mainly meant for testing things or checkout what the behavior is.
> >
> > Another topic is how to actually activate the grab (when in relative
> > mode where it is needed). You surely don't want grab the pointer on
> > hover as simply crossing the guest window will activate it.
>
> Sounds kind of useful. Why isn't it?
>
> > Grab-(+ungrab)-by-hotkey and grab-on-click is what SDL, virt-viewer &
> > friends are doing today. I think gtk should simply do the same for
> > consistency, and I don't see a need for a config option.
> >
> > cheers,
> > Gerd
>
> Aha.
> It's lack of grab on click that makes me unhappy.
> So how about doing this for 2.0?
> Drop grab on hover make grab on click the default.
OK, I'll submit a revised patch to implement grab-on-click only in
relative mode.
Takashi
next prev parent reply other threads:[~2014-04-08 9:23 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-04 10:41 [Qemu-devel] [PATCH v3 0/4] Fix relative pointer tracking on Gtk UI Takashi Iwai
2014-04-04 10:41 ` [Qemu-devel] [PATCH v3 1/4] gtk: Use gtk generic event signal instead of motion-notify-event Takashi Iwai
2014-04-04 10:41 ` [Qemu-devel] [PATCH v3 2/4] gtk: Fix the relative pointer tracking mode Takashi Iwai
2014-04-04 10:41 ` [Qemu-devel] [PATCH v3 3/4] gtk: Remember the last grabbed pointer position Takashi Iwai
2014-04-04 10:41 ` [Qemu-devel] [PATCH v3 4/4] gtk: Add "Grab On Click" option Takashi Iwai
2014-04-07 8:07 ` Gerd Hoffmann
2014-04-07 8:21 ` Takashi Iwai
2014-04-07 20:38 ` Michael S. Tsirkin
2014-04-08 6:25 ` Gerd Hoffmann
2014-04-08 8:11 ` Michael S. Tsirkin
2014-04-08 8:49 ` Gerd Hoffmann
2014-04-08 8:57 ` Michael S. Tsirkin
2014-04-08 9:23 ` Takashi Iwai [this message]
2014-04-04 23:08 ` [Qemu-devel] [PATCH v3 0/4] Fix relative pointer tracking on Gtk UI Brian Jackson
2014-04-06 10:36 ` Michael S. Tsirkin
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=s5hbnwcxjih.wl%tiwai@suse.de \
--to=tiwai@suse.de \
--cc=aliguori@amazon.com \
--cc=crobinso@redhat.com \
--cc=kraxel@redhat.com \
--cc=mst@redhat.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).