All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Stefan Weil <sw@weilnetz.de>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Alex Graf <agraf@suse.de>, Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 0/8] Add GTK UI to enable basic accessibility (v2)
Date: Mon, 27 Feb 2012 07:50:38 -0600	[thread overview]
Message-ID: <4F4B8A2E.1030209@codemonkey.ws> (raw)
In-Reply-To: <4F4B8858.7060202@siemens.com>

On 02/27/2012 07:42 AM, Jan Kiszka wrote:
> On 2012-02-27 14:33, Anthony Liguori wrote:
>>> That all keyboard inputs are grabbed when the mouse is in the window and
>>> you don't need to press ctrl-alt-g explicitly. And the reverse should
>>> happen when the mouse reaches the window border. Just like under SDL,
>>> give it a try.
>>
>> Right, this was intentional.  I think it's weird that a window would steal input
>> when the mouse moves over it breaking desktop accelerators like alt tab.  If
>> you've ever alt-tab cycled through windows when QEMU is running, if you're
>> unlucky enough to have the mouse where a QEMU window may be, the QEMU instance
>> will steal input and prevent alt-tab from working anymore.
>
> This would be a usability regression. It is very unhandy to switch
> between grabbed and ungrabbed, specifically for alt-tab, when working
> with guests vs. host windows. Look e.g. at how rdeskop works in this
> regard. That's why I introduced this feature to QEMU, and I would not
> want to see it die again with the introduction of GTK.

I'll add a "Grab on Hover" menu item to enable the behavior.  It's a good excuse 
to look at gconf to store GUI preferences too.

>>>>> - window not resizable (except in broken full-screen mode)
>>>>
>>>> That's intentional.
>>>
>>> And a mistake IMHO. Definitely for the text consoles, but one can also
>>> argue about the guest graphic console. I think Stefano once introduced
>>> this for some Xen use case, maybe he can comment on it.
>>
>> If we added resize, I wouldn't want to scale-to-size.  I find this behavior to
>> be more trouble than it's worth.  I'd want to render the VGA screen with a black
>> border only scaling based on the zoom settings.
>
> OK for aspect-ratio-correct scaling of the guest view from my POV. And
> if there is a use case for the old behavior, we could still add a switch
> to the menu for selecting the scaling mode.

Okay, I'll look into it.

>>> BTW, "VGA" is the wrong term for the graphic console. Maybe there is the
>>> real front-end name available somewhere and could be used instead.
>>
>> Any suggestions?  Display?  Graphics?
>>
>> Or were you thinking something like Cirrus VGA?
>
> VGA is (mostly) x86. Also, we may once have multiple guest screens,
> maybe even multiple types of them. So picking up some telling front-end
> name would likely scale best.

Display 0?

I think Monitor would be more natural but obviously that would conflict with the 
human monitor.

Regards,

Anthony Liguori

> Jan
>

  reply	other threads:[~2012-02-27 13:50 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-26 23:46 [Qemu-devel] [PATCH 0/8] Add GTK UI to enable basic accessibility (v2) Anthony Liguori
2012-02-26 23:46 ` [Qemu-devel] [PATCH 1/8] console: allow VCs to be overridden by UI Anthony Liguori
2012-02-26 23:46 ` [Qemu-devel] [PATCH 2/8] chr: check to see if front end has registered a read function Anthony Liguori
2012-02-26 23:46 ` [Qemu-devel] [PATCH 3/8] ui: add basic GTK gui (v2) Anthony Liguori
2012-02-26 23:46 ` [Qemu-devel] [PATCH 4/8] gtk: add virtual console support (v2) Anthony Liguori
2012-02-27 10:45   ` Kevin Wolf
2012-02-26 23:46 ` [Qemu-devel] [PATCH 5/8] gtk: add support for input grabbing Anthony Liguori
2012-02-26 23:46 ` [Qemu-devel] [PATCH 6/8] gtk: add support for screen scaling and full screen (v2) Anthony Liguori
2012-02-27 20:10   ` Stefan Weil
2012-02-27 22:01     ` Anthony Liguori
2012-02-28 14:18     ` Kevin Wolf
2012-02-26 23:46 ` [Qemu-devel] [PATCH 7/8] gtk: add translation support Anthony Liguori
2012-02-27  8:32   ` Paolo Bonzini
2012-02-27 13:11     ` Anthony Liguori
2012-02-28 15:10     ` Kevin Wolf
2012-02-27 22:09   ` Stefan Weil
2012-02-26 23:46 ` [Qemu-devel] [PATCH 8/8] gtk: make default UI Anthony Liguori
2012-02-27  7:23 ` [Qemu-devel] [PATCH 0/8] Add GTK UI to enable basic accessibility (v2) malc
2012-02-27  8:21 ` Jan Kiszka
2012-02-27 13:10   ` Anthony Liguori
2012-02-27 13:26     ` Jan Kiszka
2012-02-27 13:33       ` Anthony Liguori
2012-02-27 13:42         ` Jan Kiszka
2012-02-27 13:50           ` Anthony Liguori [this message]
2012-02-27 13:54             ` Jan Kiszka
2012-02-27 16:39         ` Gerd Hoffmann
2012-02-27 16:44           ` Anthony Liguori
2012-02-27 16:54             ` Gerd Hoffmann
2012-02-27 14:24 ` Kevin Wolf
2012-02-27 14:39   ` Anthony Liguori
2012-02-27 15:03     ` Kevin Wolf
2012-03-11 17:29 ` Stefan Weil
2012-03-11 18:24   ` François Revol
2012-03-11 18:47     ` Stefan Weil
2012-03-12  2:31     ` Anthony Liguori
2012-03-12 16:39       ` François Revol
2012-03-12 17:13         ` Anthony Liguori
2012-03-12  2:30   ` Anthony Liguori

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=4F4B8A2E.1030209@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=agraf@suse.de \
    --cc=jan.kiszka@siemens.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=sw@weilnetz.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.