Re: [Qemu-devel] [PATCH 2/7] ui: add basic GTK gui (v2)

[Anthony Liguori]

Stefan Weil <address@hidden> writes:

> Am 05.09.2012 21:18, schrieb Anthony Liguori:
>> This is minimalistic and just contains the basic widget infrastructure.  The 
>> GUI
>> consists of a menu and a GtkNotebook.  To start with, the notebook has its 
>> tabs
>> hidden which provides a UI that looks very similar to SDL with the exception 
>> of
>> the menu bar.
>>
>> The menu bar allows a user to toggle the visibility of the tabs.  Cairo is 
>> used
>> for rendering.
>>
>> I used gtk-vnc as a reference.  gtk-vnc solves the same basic problems as 
>> QEMU
>> since it was originally written as a remote display for QEMU.  So for the 
>> most
>> part, the approach to rendering and keyboard handling should be pretty solid 
>> for
>> GTK.
>>
>> Signed-off-by: Anthony Liguori<address@hidden>
>> ---
>> v1 ->  v2
>>   - add gtk-vnc license
>>   - fix key propagation
>> ---
>>   Makefile         |    2 +
>>   configure        |   25 +++-
>>   console.h        |    4 +
>>   sysemu.h         |    1 +
>>   ui/Makefile.objs |    1 +
>>   ui/gtk.c         |  572 
>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   6 files changed, 604 insertions(+), 1 deletions(-)
>>   create mode 100644 ui/gtk.c
>>
>
> Hi Anthony,
>
> I tested the previous version of your series and noticed that the
> menu accelerators (especially for file/quit) which are still part of
> this new patch are a bad idea.
>
> Just run a system emulation, start some GUI program in the emulation
> and press Ctrl-Q: you won't terminate the GUI program, but terminate
> QEMU!

How is that different from ctrl-alt-f or any of the other accelerators
we use?

If the argument is that Ctrl-Q is too common, that's one thing, but
we can't avoid using accelerators.

Regards,

Anthony Liguori

>
> Cheers,
>
> Stefan W.