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 696835 - Use correctly sized fedora/ubuntu logos
Use correctly sized fedora/ubuntu logos
Status: RESOLVED FIXED
Product: gnome-boxes
Classification: Applications
Component: general
3.8.x
Other Linux
: Normal normal
: 3.22
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks: 696832
 
 
Reported: 2013-03-29 09:39 UTC by Christophe Fergeau
Modified: 2016-03-31 13:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use correctly sized logos (1.03 KB, patch)
2013-04-03 17:05 UTC, Zeeshan Ali
committed Details | Review
logo: Ensure new Fedora logo gets used (1.08 KB, patch)
2013-04-03 19:53 UTC, Zeeshan Ali
committed Details | Review

Description Christophe Fergeau 2013-03-29 09:39:46 UTC
The ubuntu and fedora logos are a bit bigger than the CD icon in the media list, Jimmac was kind enough to make properly sized logos, they are at http://jimmac.fedorapeople.org/gnome3/boxes/logos/
Would be great to use these instead of the ones we are currectly using (for the ubuntu logo, the gnome-boxes-logos-db.xml will need to be updated as well).
Zeeshan, can you upload these logos to http://people.gnome.org/~zeeshanak/logos ?
Thanks!
Comment 1 Zeeshan Ali 2013-04-03 17:05:30 UTC
Created attachment 240506 [details] [review]
Use correctly sized logos
Comment 2 Christophe Fergeau 2013-04-03 17:46:31 UTC
Review of attachment 240506 [details] [review]:

ACK, wondering if the README.logos file needs changes or not (I'm not sure after looking at it)
Comment 3 Zeeshan Ali 2013-04-03 19:29:08 UTC
Attachment 240506 [details] pushed as e86caf6 - Use correctly sized logos
Comment 4 Christophe Fergeau 2013-04-03 19:35:29 UTC
Oh, missed a question I wanted to ask, I guess you updated the fedora logo in place, will the new one be downloaded as expected on systems that have already cached it?
Comment 5 Zeeshan Ali 2013-04-03 19:46:26 UTC
(In reply to comment #4)
> Oh, missed a question I wanted to ask, I guess you updated the fedora logo in
> place, will the new one be downloaded as expected on systems that have already
> cached it?

Actually no. I'll fix this..
Comment 6 Zeeshan Ali 2013-04-03 19:53:00 UTC
Created attachment 240539 [details] [review]
logo: Ensure new Fedora logo gets used

I had replaced fedora logo in-place and since we only look for existance
of file by the same name in cache, downloader will not download/refresh
the logo if the previous version has been downloaded once.

Now i gave the new logo file a different name and updating it also in
our logo db.
Comment 7 Christophe Fergeau 2013-04-03 20:01:25 UTC
Review of attachment 240539 [details] [review]:

s/existance/existence/
s/i gave/I gave/
s/updating/I updated/
Comment 8 Zeeshan Ali 2013-04-03 20:23:55 UTC
Attachment 240539 [details] pushed as 521c2b4 - logo: Ensure new Fedora logo gets used