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 553458 - [PATCH] Aisleriot, Freecell svg icons have black box artifact
[PATCH] Aisleriot, Freecell svg icons have black box artifact
Status: RESOLVED FIXED
Product: gnome-games-superseded
Classification: Deprecated
Component: general
2.24.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-games-aisleriot-maint
GNOME Games maintainers
Depends on:
Blocks:
 
 
Reported: 2008-09-23 20:00 UTC by Michael Terry
Modified: 2012-01-31 23:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (2.49 KB, patch)
2008-09-23 20:01 UTC, Michael Terry
committed Details | Review

Description Michael Terry 2008-09-23 20:00:51 UTC
On my system (Ubuntu hardy), the gnome-aisleriot.svg and gnome-freecell.svg icons have odd black box artifacts in the top right of the icon when made into a PNG file.

That is, I can open the files in an svg editor like inkscape, and they look fine.  But when I look at the scalable folder in nautilus, the thumbnails have this odd black box in the upper right.

Seems to be some bug with librsvg?

By process of elimination, I found that there is a <flowRoot> element causing the artifact.  It doesn't seem to do anything positive -- that is, removing it didn't cause any problems.

I've attached a patch that removes them.  Were they intentional?
Comment 1 Michael Terry 2008-09-23 20:01:45 UTC
Created attachment 119247 [details] [review]
Proposed patch

Copyright Canonical, Ltd
Comment 2 Christian Persch 2008-09-24 13:20:41 UTC
Those icons came from http://gnome-look.org/content/show.php/gnome-games+icons?content=52906 via bug 438032.

With the patch attached, do the icons render pixel-identical at, say, 48, 64 and 128 px size?

Since you think it's an librsvg problem, have you also filed a librsvg bug for this?

(In reply to comment #1)
> Copyright Canonical, Ltd

A bit daring to claim copyright on a trivial patch that only removes lines... :) 

Comment 3 Michael Terry 2008-09-24 13:50:32 UTC
> With the patch attached, do the icons render pixel-identical at, say, 48, 64 and 128 px size?

Yes, using inkscape.

> Since you think it's an librsvg problem, have you also filed a librsvg bug for
this?

No, I haven't.  My claim of a librsvg bug was wild speculation, and I've not traced this issue down.

> A bit daring to claim copyright on a trivial patch that only removes lines...
:)

::shrug:: A matter of habit.  I wouldn't call it daring, just at worst unnecessary.  Even if the patch is so trivial as to not be a copyrightable work, it's still appropriate to say that this patch does not come from me the person, but me the employee.  It'd be nice if Bugzilla let me put on different 'hats' and submit comments from different email addresses (without splitting my personality into two different accounts).
Comment 4 Christian Persch 2008-09-29 17:36:36 UTC
Please commit to trunk and gnome-2-24.
Comment 5 Michael Terry 2008-10-04 16:15:31 UTC
Committed to both.  Thanks!
Comment 6 Robert Ancell 2012-01-31 23:18:25 UTC
This bug is being reassigned to the "general" component so we can close the aisleriot bugzilla component.  Apologies for the mass email!