use element in SVG does not render after parent is displayed off and on

• Component: SVG
• Status: RESOLVED FIXED

[Jean-Pierre Fiset]

In SVG, if the parent element of a use element is hidden (via display="none") and then made visible again (via display="inline"), the content of the use element fails to render. A similar test case, where the actual elements are used instead of <use> elements, behaves correctly.

This problem appears on Safari 3.0.3 and on WebKit night r24944. This problem was also reproduced on MacIntel and Windows XP.

[Jean-Pierre Fiset]

Created attachment 15909 [details]
Simple Test Case

[Rob Buis]

Created attachment 16000 [details]
First attempt

This should fix it :)
Cheers,

Rob.

[Oliver Hunt]

Comment on attachment 16000 [details]
First attempt

Could you use a rect instead of circle in the test case?

otherwise r=me

[Rob Buis]

(In reply to comment #1)
> Created an attachment (id=15909) [edit]
> Simple Test Case

I forgot to say, these test cases are excellent, keep up the good reports!
Cheers,

Rob.

[Rob Buis]

Landed in r25123.

[Jean-Pierre Fiset]

(In reply to comment #3)
> (From update of attachment 16000 [details] [edit])
> Could you use a rect instead of circle in the test case?
> 
> otherwise r=me
> 

Would you like me to submit a new test case or are you stating that, in the future, I should submit test cases with rectangles as opposed to circles?

What are the ramifications of using <rect> over <circle> in test cases?

[Jean-Pierre Fiset]

(In reply to comment #4)
> (In reply to comment #1)
> > Created an attachment (id=15909) [edit]
> > Simple Test Case
> 
> I forgot to say, these test cases are excellent, keep up the good reports!
> Cheers,
> 
> Rob.
> 

Some credit is due to the organization that pays me to develop an SVG-enabled interactive atlas framework. See <http://nunaliit.org>

Thanks for noticing.

JP

[Oliver Hunt]

Jean-Pierre, it just makes the pixel tests a bit more robust across platforms, my comment was regarding the automated test that Rob included in his patch :D

[Oliver Hunt]

Jean-Pierre, so your test was perfect for the purpose of finding and fixing the source of the bug :D

[Rob Buis]

Landed in r25123.