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 350894 - Tango Icon
Tango Icon
Status: RESOLVED FIXED
Product: gparted
Classification: Other
Component: application
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gparted maintainers alias
gparted maintainers alias
: 573648 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-08-11 13:52 UTC by Tim Fuchs
Modified: 2009-03-01 17:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
my take on an icon (21.82 KB, image/svg+xml)
2006-08-11 13:56 UTC, Tim Fuchs
  Details
gparted tango (29.32 KB, application/x-gzip)
2007-05-22 10:20 UTC, Sebastian Kraft
  Details
gparted tango slight mod no1 (29.61 KB, application/x-gzip)
2007-05-23 07:30 UTC, Sebastian Kraft
  Details
gparted tango slight mod no2 (29.50 KB, application/x-gzip)
2007-05-23 07:31 UTC, Sebastian Kraft
  Details
Patch (1.85 KB, patch)
2008-09-17 12:59 UTC, Michael Monreal
none Details | Review
data dir (30.27 KB, application/zip)
2008-09-17 13:03 UTC, Michael Monreal
  Details

Description Tim Fuchs 2006-08-11 13:52:49 UTC
GParted Shoulkd have a fancy tango icon since it adds to a much more consisten look of the desktop.
Comment 1 Tim Fuchs 2006-08-11 13:56:17 UTC
Created attachment 70716 [details]
my take on an icon

This is my take on an GParted tango icon and my first tango icon ever.

It is a slightly modified version of a tango stock icon.
Comment 2 Plors (Bart H) 2006-08-25 08:59:21 UTC
looks cool, thanks :)
Comment 3 Michael Monreal 2006-12-02 11:58:09 UTC
The partition diagram on top of the HDD looks too flat IMHO. Also, is the HDD itself based off the icon from the Tango base theme? This would probably generate licensing issues.

Also, the icon should be available in all sizes (16, 22, 24, 32 pixel bitmap as well as scalable vector svg)

Proposing gparted for on of the next Tango Fridays¹

[1] http://tango.freedesktop.org/Tango_Fridays
Comment 4 Plors (Bart H) 2006-12-02 13:16:50 UTC
thanks and please keep us posted :)
Comment 5 Plors (Bart H) 2007-01-07 15:13:12 UTC
any news on this one?
Comment 6 Sebastian Kraft 2007-05-22 10:20:17 UTC
Created attachment 88588 [details]
gparted tango

what do you think?
Comment 7 Michael Monreal 2007-05-22 17:42:30 UTC
The light gray part (bottom right is too much like the HDD itself IMHO.

Otherwise, nice!
Comment 8 Sebastian Kraft 2007-05-23 07:30:13 UTC
Created attachment 88650 [details]
gparted tango slight mod no1

you can see a preview of the versions by following this link http://eska-art.de/blog/wp-content/uploads/2007/05/gparted_tango_varianten.png
Comment 9 Sebastian Kraft 2007-05-23 07:31:24 UTC
Created attachment 88651 [details]
gparted tango slight mod no2

for preview see the previous post
Comment 10 Michael Monreal 2007-05-23 08:20:12 UTC
I suggest sending those icons to the Tango mailing list and/or visiting #tango on freenode, to let the Tango artists review the icons.
Comment 11 Sebastian Kraft 2007-05-23 09:12:37 UTC
(In reply to comment #10)
> I suggest sending those icons to the Tango mailing list and/or visiting #tango
> on freenode, to let the Tango artists review the icons.
> 

i'm on #tango (needcoffee)...the artists see the 1st version and i tell them that i upload it to bugzilla to see what the devs think :)
Comment 12 Rodney Dawes 2007-08-19 16:53:07 UTC
I would lighten up the harddisk portion a bit. It's a little too black, and causes some useful contrast to be lost between the circle graph and the disk.
Comment 13 Andreas Nilsson 2008-09-16 13:09:46 UTC
Wow, this one looks great, amazing work as usual Sebastian!
Do we need a patch to install it properly as well?
Comment 14 Michael Monreal 2008-09-17 12:59:55 UTC
Created attachment 118873 [details] [review]
Patch

Small patch to make use of the themed app icon from hicolor (launcher, window icons, about dialog)
Comment 15 Michael Monreal 2008-09-17 13:03:32 UTC
Created attachment 118874 [details]
data dir

data/icon folder to be used with the patch above. This uses the newest (very nice!) version of the icons posted to the tango mailing list.

Extract it in the gparted/ root and remove the pixmaps/ folder.

Note: the .desktop file can also be moved to data/ to match other gnome project layouts but I tried to keep the changes to a minimum right now.
Comment 16 Curtis Gedak 2008-09-22 16:21:07 UTC
Thank you for the patch Michael.

My understanding of how gnome themes are supported is minimal.

Would you be able to point me to some documentation regarding how themed icons should be supported?
Comment 17 Michael Monreal 2008-09-22 19:30:23 UTC
Ok, long story short:

Generally we follow the fdo icon theme spec [1]. There's two cases:

a) launcher/window/logo icons: those often went as a single-size icon into share/pixmaps/$appname in the past. We now put them in various sizes into the apps context of the global hicolor theme (share/icons/hicolor/$size/apps/$appname). The hicolor theme acts as a fallback, so even if you use the gnome icon theme something like that on your desktop, the icons from hicolor will work.

b) app-specific icons: icons for special toolbar icons etc (not part of gtk stock or gnome-icon-theme), those were mostly installed into share/pixmaps/$appname/ or share/$appname in the past. Now, we put them into a private hicolor dir (share/$appname/icons/hicolor) and register share/$appname/icons as an icon path. See [2].

In the app itself we just use the icon by name (instead of manually creating a pixmap or loading the icon using a hardcoded path etc).

In gparted, only a) is used right now btw, the patch should take care of this. Extending this to also do b) would be easy in the future (e.g. new icons need to be added to the data/icons/ dir and listed in the "private" section of the makefile, plus a call of gtk_icon_theme_append_search_path somewhere as described in [2].

[1] http://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html
[2] http://live.gnome.org/ThemableAppSpecificIcons
Comment 18 Curtis Gedak 2008-09-22 22:47:46 UTC
Michael,

Thank you for the explanation and links.  And of course for the patch :-)

This change has been applied to the gnome SVN repository for inclusion in the next release of GParted (0.4.0).


Sebastian,

   Thank you for the cool icons.

   Would you be agreeable to having the icon, your name, and your email address added to the GParted Artwork page?
   http://gparted.sourceforge.net/artwork.php

Regards,
Curtis Gedak
Comment 19 Sebastian Kraft 2008-09-23 14:52:42 UTC
(In reply to comment #18)
> 
> Sebastian,
> 
>    Thank you for the cool icons.
>

My pleasure.

>    Would you be agreeable to having the icon, your name, and your email address
> added to the GParted Artwork page?
>    http://gparted.sourceforge.net/artwork.php
> 

Sure, that would be fine.

> 
best,

Sebastian
Comment 20 Curtis Gedak 2008-09-23 16:15:48 UTC
Thank you for the quick response Sebastian.

The GParted Artwork page has been updated.

Closing this bug.
Comment 21 Curtis Gedak 2009-03-01 17:43:30 UTC
*** Bug 573648 has been marked as a duplicate of this bug. ***