[Bug 702423] Re: Scour corrupts deja-dup.svg

[Jan Thor]

The problem with removeDuplicateGradients is the comparison of two
gradient stops, which doesn’t look at the style attribute. So for scour,
two stops might look the same, although they specify completely
different colors (or opacities) within their style attribute. Since
Inkscape stores stop colors in the style attribute, without converting
styles to XML attributes, scour will consider each and every two-stop
gradient of Inkscape as one and the same, and will remove all but the
first (unless you told it to unpack styles first). A quick fix would be
to include the style attribute in the list of attributes to look at when
deciding whether two gradient stops are identical. That is, replace the
line

        for attr in ['offset', 'stop-color', 'stop-opacity']:

with the line

        for attr in ['offset', 'stop-color', 'stop-opacity', 'style']:

A more ambitious fix would be to scan the style attribute for the
relevant information. It seems a similar problem could occur in
removeDuplicateGradientStops, which also doesn’t check the style
attribute to determine whether two stops are identical.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/702423

Title:
  Scour corrupts deja-dup.svg

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs