Bug 171964 - app icon(s) should follow fdo icon spec
Summary: app icon(s) should follow fdo icon spec
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gnumeric
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-10-28 13:15 UTC by Rex Dieter
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-01-21 01:59:12 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Rex Dieter 2005-10-28 13:15:40 UTC
In order to follow the freedesktop.org Icon spec, gnumeric's icon(s) (currently
only a 48x48 icon) should be installed to
%%_datadir/icons/hicolor/48x48/apps/gnumeric.png

gnumeric.desktop should refer to it as:
Icon=gnumeric

And gnumeric.spec should include:
%post
touch --no-create %{_datadir}/icons/hicolor ||:
gtk-update-icon-cache -q %{_datadir}/icons/hicolor 2> /dev/null ||:

%postun
touch --no-create %{_datadir}/icons/hicolor ||:
gtk-update-icon-cache -q %{_datadir}/icons/hicolor 2> /dev/null ||:


Also, /usr/share/pixmaps is for legacy ( <= fc3) desktops, and could probably be
dropped in newer versions (targetted for fc4+)

Comment 1 Hans de Goede 2005-11-30 19:05:57 UTC
This has been fixed in gnumeric-1.6.1-1 which should show up on a mirror near
you soon.

Comment 2 Brian Pepple 2006-01-20 20:53:52 UTC
The desktop file needs to be corrected for this change.

Comment 3 Brian Pepple 2006-01-21 01:59:12 UTC
Disregard comment #2, looks like it took a while for the icon cache to refresh.

Comment 4 Hans de Goede 2006-01-21 08:10:35 UTC
Actually you did find a bug, the %post script of the rpm should call
gtk-update-icon-cache

This has been fixed in CVS, I'll build it a soon as the buildsys is back up.


Comment 5 Hans de Goede 2006-01-24 11:12:20 UTC
FYI A new package which does call gtk-update-icon-cache in %post has been build.


Note You need to log in before you can comment on or make changes to this bug.