Panel always chooses 16x16 icon for KDE apps

Bug #34774 reported by Brent Cook
12
Affects Status Importance Assigned to Milestone
GNOME Panel
Fix Released
Low
gnome-panel (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

When a KDE app appears on the Applications menu, it is rendered with the 16x16 application icon, even if higher res icons appear in /usr/share/icons/hicolor.

However, adding a Custom Application Launcher item to the panel, specifying a KDE application and not setting the icon manually results in the panel displaying a higher-resolution icon (looks 32x32)

Tags: iso-testing
Revision history for this message
Brent Cook (bcook) wrote : Example

This shows several kde apps on the panel; the left icon was created with a Custom Application Launcher, the right icon was the result of dragging an app from the Applications menu.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for your bug. The 22x22 variant is used instead of the 24x24 variant. According to a discussion on IRC some days ago:
"dobey: GTK+ isn't handling Threshold properly for close-range sizes like 22 vs. 24"

Vincent, is gnome-panel asking for a 24x24 icon for 24 pixels panel? Have you an opinion on if we should use "Type=Fixed" for the 22x22 categories by example?

Changed in gnome-panel:
assignee: nobody → desktop-bugs
status: Unconfirmed → Confirmed
Revision history for this message
Vincent Untz (vuntz) wrote :

I think you already fixed this, but anyway, there are good chances that gnome-panel is asking for a 24x24 icon for 24 pixels panel. And if it doesn't, it will for a 25px or 26px panel.

I discussed with lapo about this, and he's right that adding padding in the panel should be okay. I have to try this.

Revision history for this message
Vincent Untz (vuntz) wrote :

Forgot: feel free to open a bug upstream :-)

Revision history for this message
Sebastien Bacher (seb128) wrote :
Changed in gnome-panel:
status: Unconfirmed → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

fixed upstream

Changed in gnome-panel:
status: Confirmed → Fix Committed
Changed in gnome-panel:
status: Confirmed → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

That upload fixes the bug:

 gnome-panel (2.17.90-0ubuntu1) feisty; urgency=low
 .
   * New upstream version:
     Panel:
     - Remove code warning
     - Change text of some menu items
     - Use GTK_STOCK_MISSING_IMAGE when the icon of a launcher can't be
       found and remove useless error dialog
     - Use better filenames when creating launchers
     - Simplify system menu code thanks to new control center shell
     - Fix "Add to panel" dialog to correctly display capplets
     - Migrate settings menu buttons to a control center shell launcher
     - Use 22 as default icon sizes for menus
     - Force size of launchers to 16, 22, 32 or 48
     - Use new launcher icon
     - Fix compilation on darwin
     - Fix disappearing launcher on drag and drop (and other related issues)
     - Fix transparency broken when screen size changes (<email address hidden>)
     - Don't accept every command line option, but only the deprecated
       --profile
     - Check for the logout_prompt gconf key to know whether we should show
       the logout dialog or just log out
     - Do not start dragging an applet when it's not possible because a grab
       is active, fix change of cursor icon in this case
     - Do not hardcode invisible_char in glade files
     - Correctly position applet menu in RTL environments
     - Really remove launcher file when deleting a launcher, and do it only
       when possible
     - Fix main menu not completely reloading when the menu is update
     gnome-desktop-item-edit:
     - Make it possible to create a .directory file with a name instead of
       forcing the name to $(dir)/.directory
     - Use new launcher icon
     - Make --create-new usage more logical
     libpanel-applet:
     - Do not hardcode invisible_char in glade files
     - Correctly position applet menu in RTL environments
     Clock Applet:
     - Look for evolution binary before trying to run it
     - Make sure that the last day of a multi-day events is marked in the
       calendar
     - Don't mark multiple days on the calendar if the event is an all day
       event
     - Fix compilation on some systems
     - Launch evolution when double-clicking on a task
     Fish Applet:
     - HIG fix
     - Do not hardcode invisible_char in glade files
     Notification Area Applet:
     - Add a test-tray program
     Workspace Switcher Applet:
     - Slow down speed of the spin button for number of workspaces
     Misc:
     - New launcher icon
     - Build fixes
     Documentation:
     - Add a link to the user guide
   * debian/patches/01_layout.patch:
     - updated
   * debian/patches/09_help_submenu.patch:
     - updated
   * debian/patches/09_lpi_src.patch:
     - updated
   * debian/patches/10_panel-transparency-after-resolution-change.patch:
     - dropped, fixed with the new version
   * debian/patches/11_applet_use_session_dialog.patch:
     - updated
   * debian/patches/11_panel_addto_dialog.patch:
     - updated
   * debian/patches/12_autoconf.patch:
     - updated
   * debian/patches/15_clock_start_evo_midday.patch:
     - updated

Changed in gnome-panel:
status: Fix Committed → Fix Released
Changed in gnome-panel:
importance: Unknown → Low
Revision history for this message
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/34774

tags: added: iso-testing
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.