Incorrect property inheritance

Bug #734019 reported by Louis Simard
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Scour
Fix Released
High
Patrick Storz

Bug Description

Properties seem to be inherited incorrectly in Scour. Here's the declaration in the example file gnome-dev-pcmcia.svg:

  <g
    [...] id="text5059"
    style="[...];fill:none;stroke:black;stroke-width:0.99067909px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1">
    <path
       [...] id="path3675"
       style="[...];fill:white;fill-opacity:1;stroke:none;[...]" />

But the path ends up with stroke:black due to incorrect inheritance rules. The text PCMCIA, described by the above-named path in the example file, is then black instead of white.

[Inferred information from bug 702423]
Scour version used = trunk r206
Command line used = scour.py --disable-style-to-xml -i INFILE -o OUTFILE

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

[Test case imported from bug 702423]

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

Set to Confirmed because I see what the changed declaration is.

Priority set to High because this bug corrupts documents.

Changed in scour:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Vish (vish) wrote :

In addition to the gnome-dev-pcmcia.svg, i found 3 other icons which were similarly affected:
/../devices/48/gnome-dev-jazdisk.svg
/../devices/48/gnome-dev-media-cf.svg
/../devices/48/gnome-dev-zipdisk.svg

All of them with text problems.

Revision history for this message
Vish (vish) wrote :
Revision history for this message
Patrick Storz (ede123) wrote :

Another testcase, same problem:

<g style="stroke:#000000;stroke-width:2.8">
    <path id="1" style="fill:#da121a" d="..." />
    <path id="2" style="fill:#fcdd09;stroke:none" d="..." />
  </g>

"stroke:none" is stripped from path 2, therefore it inherits the black stroke from the parent.

Revision history for this message
jazzynico (jazzynico) wrote :
Revision history for this message
Patrick Storz (ede123) wrote :

Fixed in scour 0.35 [1]

See copy of this bug on GitHub for details
https://github.com/scour-project/scour/issues/71

[1] https://pypi.python.org/pypi/scour

Changed in scour:
status: Confirmed → Fix Released
assignee: nobody → Eduard Braun (eduard-braun2)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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