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 719941 - Include high-resolution app icon
Include high-resolution app icon
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: ux
unspecified
Other Linux
: Normal normal
: 0.6.0
Assigned To: Geary Maintainers
Geary Maintainers
review
Depends on:
Blocks:
 
 
Reported: 2013-12-06 00:50 UTC by Jim Nelson
Modified: 2013-12-20 20:12 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
app icon: update high resolution to be detailed (331.56 KB, patch)
2013-12-06 16:20 UTC, Jakub Steiner 		none Details | Review
app icon: update high resolution to be detailed (449.83 KB, patch)
2013-12-06 16:51 UTC, Jakub Steiner 		none Details | Review

Description Jim Nelson 2013-12-06 00:50:40 UTC 
It's been requested by the GNOME project that Geary include a high-resolution application icon: https://wiki.gnome.org/action/edit/GnomeGoals/HighResolutionAppIcons

However, Geary's current icon is an SVG that scales well (at least to my eyes), so I don't know that any work has to be done yet.  I'll get in touch with the GNOME design folks and see what we need to do, if anything.
Comment 1 Jakub Steiner 2013-12-06 12:16:25 UTC 
Thanks for filing the bug, Jim. 

The (minor) issue with the current icon is that its strokes get scaled along with the icon when rendered at higher resolution making it look a little more cartoony among the other highres icons. We'll apply some trickery and provide a patch that keeps the metaphor/look intact.
Comment 2 Jakub Steiner 2013-12-06 16:20:10 UTC 
Created attachment 263680 [details] [review]
app icon: update high resolution to be detailed
Comment 3 Jakub Steiner 2013-12-06 16:51:41 UTC 
Created attachment 263684 [details] [review]
app icon: update high resolution to be detailed
Comment 4 Jim Nelson 2013-12-10 01:14:35 UTC 
Jakub, this looks fine to me.  One thing: your patch includes a new "icons/geary.svg" file that appears to have one large image and several smaller images of varying sizes.  Was this a work file that got added to the patch by mistake?  I don't believe it's installed or used anywhere, but want to double-check.
Comment 5 William Jon McCann 2013-12-19 20:23:51 UTC 
Depending on how you interpret the GPL it is required to include this "source" file for compliance. It also helps if anyone wants to make adjustments later.
Comment 6 Jim Nelson 2013-12-20 20:12:54 UTC 
Fixed in commit efd15ad90.