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 432301 - make app icon theme-friendly
make app icon theme-friendly
Status: RESOLVED FIXED
Product: dasher
Classification: Applications
Component: core
4.4.x
Other Linux
: Normal normal
: ---
Assigned To: Phil Cowans
Phil Cowans
Depends on:
Blocks:
 
 
Reported: 2007-04-22 16:08 UTC by Christian Kirbach
Modified: 2007-11-04 22:05 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18

Attachments
make app icon more theme-friendly (gnome goal 4) (1.50 KB, patch)
2007-05-02 18:32 UTC, Christian Kirbach 		none Details | Review
update: make app icon more theme-friendly (gnome goal 4) (1.50 KB, patch)
2007-10-07 10:40 UTC, Christian Kirbach 		committed Details | Review

Description Christian Kirbach 2007-04-22 16:08:02 UTC 
one of the Gnome goals. Patch will hopefully follow from me.
Comment 1 Christian Kirbach 2007-04-22 17:18:39 UTC 
as of now dasher only has icons for Microsoft platforms.
Comment 2 Luis Menina 2007-04-25 08:25:19 UTC 
Please reopen, there's at least the dasher application icon which is distributed.
You'll need to patch this file:
http://svn.gnome.org/viewcvs/dasher/trunk/Data/Makefile.am

Blocks bug 362604
Comment 3 Phil Cowans 2007-04-25 12:21:02 UTC 
We do indeed have gnome icons.
Comment 4 Christian Kirbach 2007-05-02 18:30:48 UTC 
/me probably lacked sleep ;)
Comment 5 Christian Kirbach 2007-05-02 18:32:26 UTC 
Created attachment 87404 [details] [review]
make app icon more theme-friendly (gnome goal 4)

also updates icon cache when uninstalling. some adjustments to make naming more consistent troughout Gnome apps.
Comment 6 Christian Kirbach 2007-10-07 10:40:00 UTC 
Created attachment 96809 [details] [review]
update: make app icon more theme-friendly (gnome goal 4)

patch updated to apply cleanly

Consider this a patch nag
Comment 7 Phil Cowans 2007-11-04 18:18:39 UTC 
Patch now applied to trunk - thanks.