Bug 18228 – SVG graphics do not actually work in IMG tags or as background-image properties

• Component: SVG
• Status: RESOLVED FIXED

[Kay Summers]

According to this document: http://docs.info.apple.com/article.html?artnum=307467 I should be able to display SVG graphics with simple IMG tags as well as in CSS properties such as background-image. In practice, this does not actually seem to work.

A combined test case of several ways I've found SVG support to be broken is available here:

http://www.khiltd.com/svg.html

[Mark Rowe (bdash)]

Which of the examples do not work?  They all appear correctly for me in Safari 3.1, and a recent nightly build.

[Eric Seidel (no email)]

I'm not sure that all cases work in 3.1.

[Marco Barisione]

(In reply to comment #0)
> A combined test case of several ways I've found SVG support to be broken is
> available here:

Are you using WebKit GTK? It has problems with SVG images in <img> tags, see #19370.

[Nikolas Zimmermann]

Created attachment 95637 [details]
Patch

[Rob Buis]

Comment on attachment 95637 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=95637&action=review

LGTM

> LayoutTests/svg/custom/resources/spinner.svg:4
> +<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="40px" height="40px" viewBox="0 0 50 50" preserveAspectRatio="xMinYMin">

preserveAspectRatio needed? And not the default value used?

> LayoutTests/svg/custom/resources/spinner.svg:10
> +        <animateTransform attributeName="transform" type="rotate" values="30,25,25; 60,25,25; 90,25,25; 120,25,25; 150,25,25; 180,25,25; 210,25,25; 240,25,25; 270,25,25; 300,25,25; 330,25,25; 0,25,25;" dur="1s" repeatCount="indefinite" calcMode="discrete" />

Line can be removed

[Nikolas Zimmermann]

(In reply to comment #5)
> (From update of attachment 95637 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=95637&action=review
> 
> LGTM
> 
> > LayoutTests/svg/custom/resources/spinner.svg:4
> > +<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="40px" height="40px" viewBox="0 0 50 50" preserveAspectRatio="xMinYMin">
> 
> preserveAspectRatio needed? And not the default value used?
No, will remove it.

> 
> > LayoutTests/svg/custom/resources/spinner.svg:10
> > +        <animateTransform attributeName="transform" type="rotate" values="30,25,25; 60,25,25; 90,25,25; 120,25,25; 150,25,25; 180,25,25; 210,25,25; 240,25,25; 270,25,25; 300,25,25; 330,25,25; 0,25,25;" dur="1s" repeatCount="indefinite" calcMode="discrete" />
> 
> Line can be removed

Fixed, landed in r87840.

[Nikolas Zimmermann]

*** Bug 18227 has been marked as a duplicate of this bug. ***

[Nikolas Zimmermann]

*** Bug 22986 has been marked as a duplicate of this bug. ***