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 395919 - [Patch] Tangoified icon for Meld
[Patch] Tangoified icon for Meld
Status: RESOLVED FIXED
Product: meld
Classification: Other
Component: general
git master
Other All
: Normal minor
: ---
Assigned To: Stephen Kennedy
Stephen Kennedy
Depends on:
Blocks:
 
 
Reported: 2007-01-12 20:56 UTC by Josef Vybíral
Modified: 2009-10-11 02:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Tangoified icon for Meld (23.01 KB, application/x-bzip)
2007-01-12 20:58 UTC, Josef Vybíral
  Details
Tangoified icon for Meld (23.09 KB, application/x-bzip)
2007-01-13 20:04 UTC, Josef Vybíral
  Details
changed makefile to install new icon set (925 bytes, patch)
2009-02-19 09:07 UTC, Stefan Friesel
none Details | Review

Description Josef Vybíral 2007-01-12 20:56:30 UTC
The current logo/icon does not fit well the currently "developed" default gnome-icon-theme. I created a new tangoified version of the current logo that can help to integrate Meld into the gnome menu as well as to the user's desktop.

A package with the icon is attached to this bug report.

Other information:
Comment 1 Josef Vybíral 2007-01-12 20:58:07 UTC
Created attachment 80142 [details]
Tangoified icon for Meld

Package includes all required sizes and also sources for them.
Comment 2 Josef Vybíral 2007-01-13 20:04:51 UTC
Created attachment 80197 [details]
Tangoified icon for Meld

Steven Garitty sugested to make the gloss a little bit transparent, so here is the updated version that replaces previous one. As in the previous case, all required sizes including "sources" are included in the package.
Comment 3 Stephen Kennedy 2007-02-17 18:38:44 UTC
Thanks for the icons! Can you also make a patch to use and install the new icons? 
Comment 4 Josef Vybíral 2007-02-17 21:03:34 UTC
Hi Stephen, I am afraid my artistic skills replace all my coding skills :/ All I know about the coding part of this is that icons should go to $prefix/share/icons/hicolor/<size>/apps/ and the code "how to get them" is probably the one mentioned at [1].

[1] - http://wayofthemonkey.com/?date=2006-11-15
Comment 5 Rodney Dawes 2007-08-19 16:20:45 UTC
What still needs to be done to get these icons in?
Comment 6 John Keller 2008-09-01 20:20:56 UTC
Ping, in case there's still time for this to get in to 2.24...
Comment 7 Stephen Kennedy 2008-09-03 21:31:12 UTC
I have committed the icons to svn. (probably in the wrong place, but anyway...)

The thing missing now is how to hook the icons up.

1) modify the install to put the icons in the right place
2) modify the source to find the new icons, both when installed and when run from an unpacked tarball.
Comment 8 Stefan Friesel 2009-02-19 09:07:22 UTC
Created attachment 129045 [details] [review]
changed makefile to install new icon set

I attached a patch to install the icon set (according to some other packages, they go to $(DESTDIR)$(sharedir)/icons/hicolor/{resolution}/apps/{appname.png}). I also did
cp glade2/pixmaps/32x32/meld.png glade2/pixmaps/icon.png
to replace the old one, so it also works when running from the tarball.
I also had to add
mkdir $(DESTDIR)$(sharedir_)
in the makefile as otherwise I got the error
"mkdir: could not create directory "/usr/share/meld": No such file or directory"
but this probably does no harm.
Comment 9 Vincent Legoll 2009-07-18 09:51:15 UTC
Anything more than Stephan's patch to be done ?
Can I commit this ?
Comment 10 John Keller 2009-10-10 23:47:50 UTC
Is there any way to include the SVG icon somewhere, to ensure that downstream packagers know about and use it?

Also, has the patch been commited? I see the new icon in the application (GNOME 2.28 in Mandriva 2010.0 cooker), but not in the "hicolor" directory. So all the various sizes seem to have been left out.

It seems a shame that such a nice range of optimized icons don't get used (e.g. gedit in gnome-do's docky shows up nice and smooth when the icon is magnified, but Meld is all blurry because the packagers use the smaller application icons).
Comment 11 Kai Willadsen 2009-10-11 02:45:46 UTC
I've just committed a modified version of Stefan's patch, so this bug should be fixed.

As for the SVG icon, there was no actual scalable/ folder included, so I've just set the Makefile to install the SVG version from the 48x48/ folder. If that was the wrong thing to do, please let me know.