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 645174 - New icons
New icons
Status: RESOLVED FIXED
Product: gnome-disk-utility
Classification: Core
Component: Disks UI
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-disk-utility-maint
[gnome3-important] [icon]
Depends on:
Blocks:
 
 
Reported: 2011-03-18 19:05 UTC by William Jon McCann
Modified: 2011-03-21 16:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Package with rendered icons + svg 'source' file. (331.50 KB, application/x-bzip)
2011-03-19 12:08 UTC, Lapo Calamandrei
Details

Description William Jon McCann 2011-03-18 19:05:49 UTC
Master Lapo tells me that we're missing a hires icon.
Comment 1 David Zeuthen (not reading bugmail) 2011-03-18 19:35:10 UTC
There's already a SVG of the application icon .. palimpsest.svg:

 http://git.gnome.org/browse/gnome-disk-utility/tree/data/icons/scalable

 .. isn't that good enough?
Comment 2 William Jon McCann 2011-03-18 19:44:32 UTC
No.  We need a 256px version.
Comment 3 Lapo Calamandrei 2011-03-19 12:08:36 UTC
Created attachment 183782 [details]
Package with rendered icons + svg 'source' file.

here are new icons, please install in hicolor following the structure of the package. The svg is the source file (used to render the pngs), it sholdn't be shipped with the app, but it should be kept in the repository to comply with the source clause, since the icons are licensed LGPL3/CCBYSA3 (like gnome-icon-theme icons).
Comment 4 David Zeuthen (not reading bugmail) 2011-03-20 17:02:56 UTC
(In reply to comment #3)
> Created an attachment (id=183782) [details]
> Package with rendered icons + svg 'source' file.
> 
> here are new icons, please install in hicolor following the structure of the
> package. The svg is the source file (used to render the pngs), it sholdn't be
> shipped with the app, but it should be kept in the repository to comply with
> the source clause, since the icons are licensed LGPL3/CCBYSA3 (like
> gnome-icon-theme icons).

Nice icon, but this bug is about adding a 256x256 icon of the _existing_ app icon - not replacing the icon for the application. What gives?

FWIW, I'm kinda opposed to replacing the icon since the current icon is tied to the name Palimpsest... then again, we don't really advertise the name Palimpsest anywhere any more...
Comment 5 William Jon McCann 2011-03-20 17:05:20 UTC
Yeah the Palimpsest reference doesn't really make sense any more.

Can we please use Lapo's icons?
Comment 6 David Zeuthen (not reading bugmail) 2011-03-20 17:48:30 UTC
(In reply to comment #5)
> Yeah the Palimpsest reference doesn't really make sense any more.
> 
> Can we please use Lapo's icons?

I guess. But what about the fact that we're way past UI freeze?
Comment 7 Lapo Calamandrei 2011-03-20 18:04:28 UTC
David, as noted the current icon doesn't make much sense (I even had no time to draw the current one). this one was quickly made composing existing stuff we have in G-I-T, but I think it's clearer then what is used currently.
Comment 8 William Jon McCann 2011-03-20 18:05:09 UTC
Should be fine since it doesn't negatively impact screenshots or docs.  Fixing the icons we show by default is an important goal for us for the release.  We've been fixing quite a few the past few weeks.  This is one of the last few remaining.  Sorry it took to long but I think it is worthwhile to fix.