qemu-devel.nongnu.org archive mirror
 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 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).