Re: [Qemu-devel] [PATCH] ui/gtk: Set QEMU window icon

[Stefan Weil <sw@weilnetz.de>]

[-- Attachment #1: Type: text/plain, Size: 2283 bytes --]

Am 10.04.2013 13:48, schrieb Jan Kiszka:
> On 2013-03-30 15:21, Stefan Weil wrote:
>> The QEMU icon which is already used for SDL
>> is now also loaded by GTK.
>>
>> Signed-off-by: Stefan Weil<sw@weilnetz.de>
>> ---
>>   ui/gtk.c |   13 +++++++++++++
>>   1 file changed, 13 insertions(+)
>>
>> diff --git a/ui/gtk.c b/ui/gtk.c
>> index a5a8156..c15023b 100644
>> --- a/ui/gtk.c
>> +++ b/ui/gtk.c
>> @@ -1363,6 +1363,7 @@ static const DisplayChangeListenerOps dcl_ops = {
>>   void gtk_display_init(DisplayState *ds)
>>   {
>>       GtkDisplayState *s = g_malloc0(sizeof(*s));
>> +    char *filename;
>>
>>       gtk_init(NULL, NULL);
>>
>> @@ -1394,6 +1395,18 @@ void gtk_display_init(DisplayState *ds)
>>
>>       gtk_notebook_append_page(GTK_NOTEBOOK(s->notebook), s->drawing_area, gtk_label_new("VGA"));
>>
>> +    filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, "qemu-icon.bmp");
>> +    if (filename) {
>> +        GError *error = NULL;
>> +        GdkPixbuf *pixbuf = gdk_pixbuf_new_from_file(filename,&error);
>> +        if (pixbuf) {
>> +            gtk_window_set_icon(GTK_WINDOW(s->window), pixbuf);
>> +        } else {
>> +            g_error_free(error);
>> +        }
>> +        g_free(filename);
>> +    }
>> +
>>       gd_create_menus(s);
>>
>>       gd_connect_signals(s);
>>
>
> With gtk 2.22 I'm lacking transparency of that icon (it has a white
> background). The icon is fine with SDL. Is that an issue of this
> particular gtk version or actually a general problem with
> gdk_pixbuf_new_from_file?
>
> Jan

I'm afraid that you won't get a transparent background as long
as the image format does not support transparency.

SDL only accepts BMP files and obviously translates the white background
to a transparent one.

I could have added a PNG icon for gtk, but did not do so because I
thought that the BMP file looked sufficiently good (better than the
default icon on my Debian hosts). There could be a GUI option
which allows setting an arbitrary icon file. It would allow users
to choose icons with transparency for gtk and also different icons
for multiple QEMU instances.

Stefan

PS. Try the appended qemu-icon.bmp. It won't work with SDL, but show
the desired result with gtk because it is not a BMP file, but a PNG
file with transparent background.



[-- Attachment #2: qemu-icon.bmp --]
[-- Type: image/bmp, Size: 576 bytes --]