611976 - Animate of fill property fails when animating "d" attribute of the same path

Status: RESOLVED INVALID

(Core :: SVG, defect)

Description

[Marek Raida]

User-Agent:       Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101112 Firefox/4.0b8pre
Build Identifier: 

Newly implemented animation of "d" attribute interferes somehow with existing animation of "fill" property of path...

Reproducible: Always

Steps to Reproduce:
1. Open testcase from attacment
2. Color of animated path should be interpolating/changing, but stays gray
Actual Results:  
fill stays gray

Expected Results:  
fill should change/animate

Comment 1

[Marek Raida]

Attachment: Testcase

Comment 2

[Jonathan Watt [:jwatt]]

Hi Marek. Glad to see people testing this stuff! :)

In this case you seem to have just forgotten to add the xlink:href attributes on the <animate> elements targeting fill and stroke-width. If you add that it should work.

Comment 3

[Daniel Holbert [:dholbert]]

Jonathan's right -- those animations both target their parent, which in this case is the <g> element.

You can fix the testcase by either
 (a) Adding "fill:inherit" style on the <path>
 (b) Removing the close- "/>" of "<path ... />" and adding a </path> after the animations.

Comment 4

[Daniel Holbert [:dholbert]]

Attachment: reduced testcase for opera issue

Interestingly, the testcase Marek attached "works" in Opera, but I think that's due to an Opera bug.

Here's a reduced testcase that's also broken in Opera, in the same way. (the <set> in this testcase affects the <rect>'s fill-color, when it shouldn't).

Firefox and Webkit agree on both tests here, and Opera's the odd one out.

Comment 5

[Marek Raida]

You're right and now I see my (stupid) mistake. Thank you guys for explanation...

Comment 6

[Boris Zbarsky [:bzbarsky]]

> Interestingly, the testcase Marek attached "works" in Opera, but I think that's
> due to an Opera bug.

Worth reporting to Opera and/or adding to the SVG test suite.