317232 - Enable SVG in Camino

Status: VERIFIED FIXED

(Camino Graveyard :: General, defect)

Description

[Mark Mentovai]

This came up in bug 317077 comment 4.  Gecko SVG support is still incomplete, but there are definite positives to an implementation that's integrated tightly into the browser.  Firefox 1.5 is shipping with SVG, it should be considered for Camino 1.0 too.

Comment 1

[Mark Mentovai]

Attachment: Enable SVG in Camino

Comment 2

[Mark Mentovai]

Comment on attachment 203760 [details] [diff] [review]
Enable SVG in Camino

The pbxproj changes add the following files to Camino.app/Contents/MacOS:

components/dom_svg.xpt
components/gksvgrenderer.xpt
res/svg.css

Comment 3

[Mike Pinkerton (not reading bugmail)]

i'd say go for it, we don't want to be in another situation like midas and gmail (where they sniff us wrong). 

though where do we draw the line? what about mathml? 

Comment 4

[Mike Shaver (:shaver emeritus)]

I think you don't draw a line, you use the line drawn by XULRunner and libxul and ship the same Gecko as everyone else.  We need to make sure that those defaults are better-reflected in the configure defaults (f.e., I think you still have to explicitly do --enable-canvas to get what Firefox ships), to help projects like Camino DTRT here.

Comment 5

[Brendan Eich [:brendan]]

(In reply to comment #3)
> i'd say go for it, we don't want to be in another situation like midas and
> gmail (where they sniff us wrong). 
> 
> though where do we draw the line? what about mathml? 


Wikipedia sniffs and uses MathML if it can -- why wouldn't Camino want that?

What problem is solved by liposuction?

/be

Comment 6

[Samuel Sidler (old account; do not CC)]

Brendan: Even if Camino wanted MathML, there are issues with "turning it on". That, however, is bug 190307.

Comment 7

[jonathan chetwynd]

What whippersnapper pipped me to the post?

Get it on for SVG support NOW!

Comment 8

[Mark Mentovai]

Comment on attachment 203760 [details] [diff] [review]
Enable SVG in Camino

pinkerton came up to New York just to give r+ in person.

Comment 9

[Mark Mentovai]

Checked in, trunk and branch.

Comment 10

[Samuel Sidler (old account; do not CC)]

v on branch and trunk.