Scour

Removes "dot" part of the icons

Bug #734933 reported by Vish
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Scour
Fix Released
Medium
codedread

Bug Description

(Using rev 207 from lp:scour. )

Scour removes the central "dot" from the top speaker of the rhythmbox icon.
IIRC, How I had done the icon was, I first made 1 icon and then modified for the rest of the required sizes/status.
(Though it is 12 icons with the problem, It might just be the same element that is being removed consistently)

Attaching both the original and scoured icons
/../Humanity/apps/22/rhythmbox-panel.svg
/../Humanity/apps/24/rhythmbox-panel.svg
/../Humanity/status/16/rhythmbox-notplaying.svg
/../Humanity/status/16/rhythmbox-panel.svg
/../Humanity/status/22/rhythmbox-notplaying.svg
/../Humanity/status/24/rhythmbox-notplaying.svg
/../Humanity-Dark/apps/22/rhythmbox-panel.svg
/../Humanity-Dark/apps/24/rhythmbox-panel.svg
/../Humanity-Dark/status/16/rhythmbox-notplaying.svg
/../Humanity-Dark/status/16/rhythmbox-panel.svg
/../Humanity-Dark/status/22/rhythmbox-notplaying.svg
/../Humanity-Dark/status/24/rhythmbox-notplaying.svg

Revision history for this message
Vish (vish) wrote :
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

I'm not sure if those files are the erroneous scoured results or the source, and what you expect the files to look like.

Changed in scour:
status: New → Incomplete
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Forget my last comment, I didn't see the "Original icons" directory made in my downloaded files directory, and didn't expect more files to come out of that little 2.9 KB tar.lzma.

Changed in scour:
status: Incomplete → New
Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Triaging:

The dot in your files is implemented as a path whose stroke is not none, and whose d element contains a MoveTo segment and a LineTo of 0,0.

This path is being reduced, by cleanPath, to only a MoveTo.

A solution to this problem would be to undo some code written by codedread in revision 120 to remove these empty drawing instructions, which in this case are essential, enabling them to be kept again.

Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

Priority set to Medium because this is a rare document-corrupting bug.

Changed in scour:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
codedread (codedread) wrote :

Thanks for the bug report and test files.

FYI, next time, it would be much less work on developers if you would just upload the one single SVG file that exhibits the problem in Scour instead of tarring, then zipping up in an obscure archive format not supported out of the box on some platforms.

Changed in scour:
assignee: nobody → codedread (codedread)
Revision history for this message
codedread (codedread) wrote :

Fixed in revision 209

Changed in scour:
status: Triaged → Fix Committed
Revision history for this message
Vish (vish) wrote :

Oops! I completely forgot about the archive format..
It was something I use occasionally for my personal files and dint think about others.. :-)

Thanks for letting me know about it..

Revision history for this message
Louis Simard (louis-simard-deactivatedaccount) wrote :

This bug is fixed in release 0.26 of Scour.

Changed in scour:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.