After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 169220 - different icons in window list and window selector applets
different icons in window list and window selector applets
Status: RESOLVED FIXED
Product: libwnck
Classification: Core
Component: selector
git master
Other Linux
: Normal normal
: ---
Assigned To: libwnck maintainers
libwnck maintainers
Depends on:
Blocks:
 
 
Reported: 2005-03-04 18:46 UTC by Vincent Noel
Modified: 2005-06-21 17:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot showing the different icons (6.56 KB, image/png)
2005-03-04 18:47 UTC, Vincent Noel
Details

Description Vincent Noel 2005-03-04 18:46:37 UTC
If you open a folder window with a recent nautilus, the icon shown in the window
list applet is a crisp, small-res icon. However,the icon shown in the window
selector applet is a blurry, scaled-down hi-res icon.

I think the window list applet has the right behavior. The window selector
should follow it.

For some reason, I can only trigger this bug with nautilus windows. All other
icon appear fine in both applets. The bug is obvious since this commit in nautilus :

2005-02-22  Alexander Larsson  <alexl@redhat.com>

        * src/nautilus-window.c: (nautilus_window_update_icon):
        Use names and not pixbufs for the window icons.
        This means the icons of the best size are used.
        Patch from Jaap Haitsma <jaap@haitsma.org>

Cc'ing Vincent for his incredible insights ;-)
Comment 1 Vincent Noel 2005-03-04 18:47:43 UTC
Created attachment 38263 [details]
screenshot showing the different icons
Comment 2 Vincent Noel 2005-06-21 16:58:29 UTC
I was about to re-submit this bug...

Here is the one-line patch to fix it, it makes tasklist.c and selector.c use the
same method to grab the window's icon. As the window selector applet icon does
not scale to follow the applet size (like the window list does), the mini icon
is better :

Index: libwnck/selector.c
===================================================================
RCS file: /cvs/gnome/libwnck/libwnck/selector.c,v
retrieving revision 1.6
diff -u -r1.6 selector.c
--- libwnck/selector.c  16 Jun 2005 17:32:36 -0000      1.6
+++ libwnck/selector.c  21 Jun 2005 16:56:33 -0000
@@ -158,7 +158,7 @@
   freeme2 = NULL;

   if (window)
-    pixbuf = wnck_window_get_icon (window);
+    pixbuf = wnck_window_get_mini_icon (window);

   if (!pixbuf)
     pixbuf = wnck_selector_get_default_window_icon ();
Comment 3 Vincent Untz 2005-06-21 17:42:52 UTC
Vincent: please commit this big patch ;-)
Comment 4 Vincent Noel 2005-06-21 17:55:06 UTC
It's not the size that counts ;)
Committed.