From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:41974) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S6ma3-0008Qe-Ib for qemu-devel@nongnu.org; Sun, 11 Mar 2012 13:29:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S6ma1-0003uO-CR for qemu-devel@nongnu.org; Sun, 11 Mar 2012 13:29:15 -0400 Received: from v220110690675601.yourvserver.net ([78.47.199.172]:49922) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S6ma1-0003sd-2i for qemu-devel@nongnu.org; Sun, 11 Mar 2012 13:29:13 -0400 Message-ID: <4F5CE0E4.2090902@weilnetz.de> Date: Sun, 11 Mar 2012 18:29:08 +0100 From: Stefan Weil MIME-Version: 1.0 References: <1330299995-8688-1-git-send-email-aliguori@us.ibm.com> In-Reply-To: <1330299995-8688-1-git-send-email-aliguori@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/8] Add GTK UI to enable basic accessibility (v2) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Kevin Wolf , Paolo Bonzini , qemu-devel@nongnu.org, Alex Graf Am 27.02.2012 00:46, schrieb Anthony Liguori: > I realize UIs are the third rail of QEMU development, but over the > years I've > gotten a lot of feedback from users about our UI. I think everyone > struggles > with the SDL interface and its lack of discoverability but it's worse > than I > think most people realize for users that rely on accessibility tools. > > The two pieces of feedback I've gotten the most re: accessibility are > the lack > of QEMU's enablement for screen readers and the lack of configurable > accelerators. > > Since we render our own terminal using a fixed sized font, we don't > respect > system font settings which means we ignore if the user has configured > large > print. > > We also don't integrate at all with screen readers which means that > for blind > users, the virtual consoles may as well not even exist. > > We also don't allow any type of configuration of accelerators. For > users with > limited dexterity (this is actually more common than you would think), > they may > use an input device that only inputs one key at a time. Holding down > two keys > at once is not possible for these users. > > These are solved problems though and while we could reinvent all of this > ourselves with SDL, we would be crazy if we did. Modern toolkits, like > GTK, > solve these problems. > > By using GTK, we can leverage VteTerminal for screen reader > integration and font > configuration. We can also use GTK's accelerator support to make > accelerators > configurable (Gnome provides a global accelerator configuration > interface). > > I'm not attempting to make a pretty desktop virtualization UI. Maybe > we'll go > there eventually but that's not what this series is about. > > This is just attempting to use a richer toolkit such that we can > enable basic > accessibility support. As a consequence, the UI is much more usable > even for a > user without accessibility requirements so it's a win-win. > > Also available at: > > https://github.com/aliguori/qemu/tree/gtk.2 > > --- > v1 -> v2 > - Add internationalization support. I don't actually speak any other > languages > so I added a placeholder for a German translation. This can be tested with > LANGUAGE=de_DE.UTF-8 qemu-system-x86_64 > - Fixed the terminal size for VteTerminal widgets. I think the > behavior makes > sense now. > - Fixed lots of issues raised in review comments (see individual patches) > > Known Issues: > - I saw the X crash once. I think it has to do with widget sizes. I > need to > work harder to reproduce. > - I've not recreated the reported memory leak yet. > - I haven't added backwards compatibility code for older VteTerminal > widgets > yet. Hi Anthony, are you still working on a new version of this patch series? I suggest to commit a slightly modified version of v2 which adds the GTK UI as an optional user interface (only enabled by a configure option). This makes testing easier and allows developers to send patches which improve the new UI. As soon as the GTK UI is considered stable and usable, the default could be changed from SDL to GTK. Regards, Stefan Weil PS. Of course the committed patches should pass checkpatch.pl without errors.